-
-
Notifications
You must be signed in to change notification settings - Fork 234
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
[National Highways] Initialise single-sign on #5114
base: master
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #5114 +/- ##
==========================================
+ Coverage 82.66% 83.73% +1.06%
==========================================
Files 404 408 +4
Lines 31462 35101 +3639
Branches 4988 5996 +1008
==========================================
+ Hits 26008 29391 +3383
- Misses 3975 4157 +182
- Partials 1479 1553 +74 ☔ View full report in Codecov by Sentry. |
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.
One thing that doesn't matter really if it's only for debugging
|
||
my $message = ''; | ||
for my $key (sort keys %{$id_token->payload}) { | ||
$message = $message . $key . " : " . $id_token->payload->{$key} . "\n" if $id_token->payload->{$key}; |
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.
$message .= $key...
Initial setup for single sign on
7cc0089
to
eb85622
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.
Nice!
67eadeb
to
4d79338
Compare
This reduces confusion if a browser auto-fills the email form but someone still clicks the SSO button, or similar.
4d79338
to
5c5a381
Compare
Initial setup for single sign on
Include debugging info to show payload for seeing what fields are used for this and going forward.
https://3.basecamp.com/4020879/buckets/37793327/todos/7702417410#__recording_7705494216
[skip changelog]