Fix Www-Authenticate not being read#885
Fix Www-Authenticate not being read#885Avery-Dunn merged 1 commit intoAzureAD:devfrom blancqua:blancqua/www-auth-fix
Conversation
|
@microsoft-github-policy-service agree |
| return response.headers() | ||
| .entrySet() | ||
| .stream() | ||
| .filter(entry -> WWW_AUTHENTICATE_HEADER.equalsIgnoreCase(entry.getKey())) |
There was a problem hiding this comment.
I think this warrants a unit test, just updating the existing one from here 68bba51#diff-c1e78d16db8b6cb86ef4e37c4f14c7950221b0fd619b2ab501499c588b5c0833R620
There was a problem hiding this comment.
Hi @bgavrilMS ! I extended the existing tests to run with both variations of the header key. Slipped in some small refactor as well to avoid duplication, hope you don't mind.
|
Hi @bgavrilMS ! I extended the existing tests to run with both variations of the header key. Slipped in some small refactor as well to avoid duplication, hope you don't mind. |
Avery-Dunn
left a comment
Avery-Dunn
left a comment
There was a problem hiding this comment.
Looks good to me, thanks for the help with this!
|
Thanks, any clue when there might be a release including this? |
3 participants
Fixes #884