fix rust style

This commit is contained in:
olivier
2025-07-17 14:16:20 +02:00
committed by Olivier D
parent 54cb82af3e
commit 64b3b1b2a5
5 changed files with 24 additions and 16 deletions

View File

@@ -52,10 +52,10 @@ pub use self::{
upstream_oauth2::{
ClaimsImports as UpstreamOAuth2ClaimsImports, DiscoveryMode as UpstreamOAuth2DiscoveryMode,
EmailImportPreference as UpstreamOAuth2EmailImportPreference,
ImportAction as UpstreamOAuth2ImportAction, OnConflict as UpstreamOAuth2OnConflict,
ImportAction as UpstreamOAuth2ImportAction,
OnBackchannelLogout as UpstreamOAuth2OnBackchannelLogout,
PkceMethod as UpstreamOAuth2PkceMethod, Provider as UpstreamOAuth2Provider,
ResponseMode as UpstreamOAuth2ResponseMode,
OnConflict as UpstreamOAuth2OnConflict, PkceMethod as UpstreamOAuth2PkceMethod,
Provider as UpstreamOAuth2Provider, ResponseMode as UpstreamOAuth2ResponseMode,
TokenAuthMethod as UpstreamOAuth2TokenAuthMethod, UpstreamOAuth2Config,
},
};

View File

@@ -122,10 +122,9 @@ impl ConfigurationSection for UpstreamOAuth2Config {
provider.claims_imports.localpart.on_conflict,
OnConflict::Add
) && !matches!(
provider.claims_imports.localpart.action,
ImportAction::Force | ImportAction::Require
)
{
provider.claims_imports.localpart.action,
ImportAction::Force | ImportAction::Require
) {
return annotate(figment::Error::custom(
"The field `action` must be either `force` or `require` when `on_conflict` is set to `add`",
));

View File

@@ -42,10 +42,10 @@ pub use self::{
UpstreamOAuthAuthorizationSession, UpstreamOAuthAuthorizationSessionState,
UpstreamOAuthLink, UpstreamOAuthProvider, UpstreamOAuthProviderClaimsImports,
UpstreamOAuthProviderDiscoveryMode, UpstreamOAuthProviderImportAction,
UpstreamOAuthProviderImportPreference, UpstreamOAuthProviderLocalpartPreference, UpstreamOAuthProviderOnBackchannelLogout,
UpstreamOAuthProviderOnConflict, UpstreamOAuthProviderPkceMode,
UpstreamOAuthProviderResponseMode, UpstreamOAuthProviderSubjectPreference,
UpstreamOAuthProviderTokenAuthMethod,
UpstreamOAuthProviderImportPreference, UpstreamOAuthProviderLocalpartPreference,
UpstreamOAuthProviderOnBackchannelLogout, UpstreamOAuthProviderOnConflict,
UpstreamOAuthProviderPkceMode, UpstreamOAuthProviderResponseMode,
UpstreamOAuthProviderSubjectPreference, UpstreamOAuthProviderTokenAuthMethod,
},
user_agent::{DeviceType, UserAgent},
users::{

View File

@@ -15,8 +15,8 @@ pub use self::{
DiscoveryMode as UpstreamOAuthProviderDiscoveryMode,
ImportAction as UpstreamOAuthProviderImportAction,
ImportPreference as UpstreamOAuthProviderImportPreference,
OnBackchannelLogout as UpstreamOAuthProviderOnBackchannelLogout,
LocalpartPreference as UpstreamOAuthProviderLocalpartPreference,
OnBackchannelLogout as UpstreamOAuthProviderOnBackchannelLogout,
OnConflict as UpstreamOAuthProviderOnConflict, PkceMode as UpstreamOAuthProviderPkceMode,
ResponseMode as UpstreamOAuthProviderResponseMode,
SubjectPreference as UpstreamOAuthProviderSubjectPreference,

View File

@@ -1066,7 +1066,8 @@ mod tests {
.signing_key_for_alg(&JsonWebSignatureAlg::Rs256)
.unwrap();
let header = JsonWebSignatureHeader::new(JsonWebSignatureAlg::Rs256);
let id_token = Jwt::sign_with_rng(&mut rng, header, id_token_claims.clone(), &signer).unwrap();
let id_token =
Jwt::sign_with_rng(&mut rng, header, id_token_claims.clone(), &signer).unwrap();
// Provision a provider and a link
let mut repo = state.repository().await.unwrap();
@@ -1301,7 +1302,7 @@ mod tests {
&provider,
&subject,
&id_token.into_string(),
id_token_claims
id_token_claims,
)
.await
.unwrap();
@@ -1454,7 +1455,7 @@ mod tests {
&provider,
&subject,
&id_token.into_string(),
id_token_claims
id_token_claims,
)
.await
.unwrap();
@@ -1551,7 +1552,15 @@ mod tests {
let session = repo
.upstream_oauth_session()
.complete_with_link(clock, session, &link, Some(id_token.to_owned()), Some(id_token_claims), None, None)
.complete_with_link(
clock,
session,
&link,
Some(id_token.to_owned()),
Some(id_token_claims),
None,
None,
)
.await?;
Ok((link, session))