Skip to content

Commit bfa85cd

Browse files
authored
cleanup(auth): remove bool parameter for including email claim (#3789)
Inline with changes on #3763, in which the ADC flow will have a method to just enable email claims. Towards #3449
1 parent 527c3d5 commit bfa85cd

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

src/auth/integration-tests/src/lib.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -643,7 +643,7 @@ pub mod unstable {
643643
&target_principal_email,
644644
source_sa_creds,
645645
)
646-
.with_include_email(true)
646+
.with_include_email()
647647
.build()
648648
.expect("failed to setup id token credentials");
649649

src/auth/src/credentials/idtoken/impersonated.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -144,8 +144,8 @@ impl Builder {
144144
}
145145

146146
/// Should include email claims in the ID Token.
147-
pub fn with_include_email(mut self, include_email: bool) -> Self {
148-
self.include_email = Some(include_email);
147+
pub fn with_include_email(mut self) -> Self {
148+
self.include_email = Some(true);
149149
self
150150
}
151151

@@ -428,7 +428,7 @@ mod tests {
428428
});
429429
let creds = Builder::new("test-audience", impersonated_credential)
430430
.with_delegates(vec!["delegate1", "delegate2"])
431-
.with_include_email(true)
431+
.with_include_email()
432432
.build()?;
433433

434434
let token = creds.id_token().await?;

0 commit comments

Comments
 (0)