Skip to content

Commit ea8c8d2

Browse files
committed
fix rust style
1 parent dbdaa79 commit ea8c8d2

File tree

5 files changed

+24
-16
lines changed

5 files changed

+24
-16
lines changed

crates/config/src/sections/mod.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,10 @@ pub use self::{
5252
upstream_oauth2::{
5353
ClaimsImports as UpstreamOAuth2ClaimsImports, DiscoveryMode as UpstreamOAuth2DiscoveryMode,
5454
EmailImportPreference as UpstreamOAuth2EmailImportPreference,
55-
ImportAction as UpstreamOAuth2ImportAction, OnConflict as UpstreamOAuth2OnConflict,
55+
ImportAction as UpstreamOAuth2ImportAction,
5656
OnBackchannelLogout as UpstreamOAuth2OnBackchannelLogout,
57-
PkceMethod as UpstreamOAuth2PkceMethod, Provider as UpstreamOAuth2Provider,
58-
ResponseMode as UpstreamOAuth2ResponseMode,
57+
OnConflict as UpstreamOAuth2OnConflict, PkceMethod as UpstreamOAuth2PkceMethod,
58+
Provider as UpstreamOAuth2Provider, ResponseMode as UpstreamOAuth2ResponseMode,
5959
TokenAuthMethod as UpstreamOAuth2TokenAuthMethod, UpstreamOAuth2Config,
6060
},
6161
};

crates/config/src/sections/upstream_oauth2.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -122,10 +122,9 @@ impl ConfigurationSection for UpstreamOAuth2Config {
122122
provider.claims_imports.localpart.on_conflict,
123123
OnConflict::Add
124124
) && !matches!(
125-
provider.claims_imports.localpart.action,
126-
ImportAction::Force | ImportAction::Require
127-
)
128-
{
125+
provider.claims_imports.localpart.action,
126+
ImportAction::Force | ImportAction::Require
127+
) {
129128
return annotate(figment::Error::custom(
130129
"The field `action` must be either `force` or `require` when `on_conflict` is set to `add`",
131130
));

crates/data-model/src/lib.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,10 @@ pub use self::{
4242
UpstreamOAuthAuthorizationSession, UpstreamOAuthAuthorizationSessionState,
4343
UpstreamOAuthLink, UpstreamOAuthProvider, UpstreamOAuthProviderClaimsImports,
4444
UpstreamOAuthProviderDiscoveryMode, UpstreamOAuthProviderImportAction,
45-
UpstreamOAuthProviderImportPreference, UpstreamOAuthProviderLocalpartPreference, UpstreamOAuthProviderOnBackchannelLogout,
46-
UpstreamOAuthProviderOnConflict, UpstreamOAuthProviderPkceMode,
47-
UpstreamOAuthProviderResponseMode, UpstreamOAuthProviderSubjectPreference,
48-
UpstreamOAuthProviderTokenAuthMethod,
45+
UpstreamOAuthProviderImportPreference, UpstreamOAuthProviderLocalpartPreference,
46+
UpstreamOAuthProviderOnBackchannelLogout, UpstreamOAuthProviderOnConflict,
47+
UpstreamOAuthProviderPkceMode, UpstreamOAuthProviderResponseMode,
48+
UpstreamOAuthProviderSubjectPreference, UpstreamOAuthProviderTokenAuthMethod,
4949
},
5050
user_agent::{DeviceType, UserAgent},
5151
users::{

crates/data-model/src/upstream_oauth2/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,8 @@ pub use self::{
1515
DiscoveryMode as UpstreamOAuthProviderDiscoveryMode,
1616
ImportAction as UpstreamOAuthProviderImportAction,
1717
ImportPreference as UpstreamOAuthProviderImportPreference,
18-
OnBackchannelLogout as UpstreamOAuthProviderOnBackchannelLogout,
1918
LocalpartPreference as UpstreamOAuthProviderLocalpartPreference,
19+
OnBackchannelLogout as UpstreamOAuthProviderOnBackchannelLogout,
2020
OnConflict as UpstreamOAuthProviderOnConflict, PkceMode as UpstreamOAuthProviderPkceMode,
2121
ResponseMode as UpstreamOAuthProviderResponseMode,
2222
SubjectPreference as UpstreamOAuthProviderSubjectPreference,

crates/handlers/src/upstream_oauth2/link.rs

Lines changed: 13 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1066,7 +1066,8 @@ mod tests {
10661066
.signing_key_for_alg(&JsonWebSignatureAlg::Rs256)
10671067
.unwrap();
10681068
let header = JsonWebSignatureHeader::new(JsonWebSignatureAlg::Rs256);
1069-
let id_token = Jwt::sign_with_rng(&mut rng, header, id_token_claims.clone(), &signer).unwrap();
1069+
let id_token =
1070+
Jwt::sign_with_rng(&mut rng, header, id_token_claims.clone(), &signer).unwrap();
10701071

10711072
// Provision a provider and a link
10721073
let mut repo = state.repository().await.unwrap();
@@ -1301,7 +1302,7 @@ mod tests {
13011302
&provider,
13021303
&subject,
13031304
&id_token.into_string(),
1304-
id_token_claims
1305+
id_token_claims,
13051306
)
13061307
.await
13071308
.unwrap();
@@ -1454,7 +1455,7 @@ mod tests {
14541455
&provider,
14551456
&subject,
14561457
&id_token.into_string(),
1457-
id_token_claims
1458+
id_token_claims,
14581459
)
14591460
.await
14601461
.unwrap();
@@ -1551,7 +1552,15 @@ mod tests {
15511552

15521553
let session = repo
15531554
.upstream_oauth_session()
1554-
.complete_with_link(clock, session, &link, Some(id_token.to_owned()), Some(id_token_claims), None, None)
1555+
.complete_with_link(
1556+
clock,
1557+
session,
1558+
&link,
1559+
Some(id_token.to_owned()),
1560+
Some(id_token_claims),
1561+
None,
1562+
None,
1563+
)
15551564
.await?;
15561565

15571566
Ok((link, session))

0 commit comments

Comments
 (0)