-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Google auth 240 #136
Google auth 240 #136
Conversation
94e83b3
to
90ee484
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does what it says on the tin
GoogleAuthConfig( | ||
clientId, | ||
clientSecret, | ||
redirectUrl, | ||
domain, | ||
domains = List("guardian.co.uk"), | ||
maxAuthAge = Some(java.time.Duration.ofDays(90)), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like quite a long time?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah... bonono is set to the same value.
Happy to find a better example in another tool and then adjust these in 2 new PRs.
What does this change?
Update to latest google auth library and drop unused overrides.
Likely enforces 2FA.
We should be trying to drop as many overrides as we can as the frameworks we depend on catch up with better dependencies of their own.
How to test
How can we measure success?
Have we considered potential risks?
Images
Accessibility