Skip to content

[GH-62] add a supported AuthnContextClass so the users with "fido" authenticaion at FredHutch can authenticate to our Service Provider (aka Relying Party) - #63

Merged
goulter merged 2 commits into
mainfrom
GH-62
Aug 5, 2026
Merged

[GH-62] add a supported AuthnContextClass so the users with "fido" authenticaion at FredHutch can authenticate to our Service Provider (aka Relying Party)#63
goulter merged 2 commits into
mainfrom
GH-62

Conversation

@goulter

@goulter goulter commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Change Description: Some FHCC users are receiving this error upon 2fa:
image

https://teams.microsoft.com/l/message/19:b02654641e514c60a13851833efd626a@thread.tacv2/1785949564317?tenantId=f6b6dd5b-f02f-441a-99a0-162ac5060bd2&groupId=7b584b6a-8636-41cb-9ddf-7a896caeb827&parentMessageId=1785949564317&teamName=UWIT%20Incident%20and%20Service%20Desk%20Coordination&channelName=Identity%20and%20Access%20Management%20(IAM)&createdTime=1785949564317

Closes GH(s): GH-62

Entra ID SAML authnmethodsreferences (AMR) now supports phishing-resistant MFA
AuthnContextClassRef: urn:oasis:names:tc:SAML:2.0:ac:classes:SmartcardPKI

Single sign-on SAML (Security Assertion Markup Language) protocol - Microsoft identity platform
image

uw-saml-python Pull Request checklist

  • I have run ./scripts/pre-push.sh
  • I have selected a semver-guidance: label for this pull request (under labels,
    to the right of the screen)

If you do not do both of these things, your checks will either not run, or have a high probability of failing.

…tion at FredHutch can authenticate to our Service Provider (aka Relying Party)
@goulter goulter self-assigned this Aug 5, 2026
@goulter
goulter requested review from jdiverp and jprosser August 5, 2026 22:40
@jdiverp

jdiverp commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

lets give this a try

@jprosser

jprosser commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

you may be able to fix this more permenately by just deleting the request for MFA and leaving it up to the FH IdP to decide here:

f"{_attribute_prefix2}/authnmethodsreferences": "saml_authncontextclassref",

@goulter

goulter commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

@jprosser I'm not the expert for sure, but I like your suggestion. If we want FH to take ownership, like you suggest, we probably need to coordinate that kind of change with them. Then again, I might be a little green here 😅

@goulter

goulter commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

I'll merge this in.

@goulter
goulter merged commit 78e9d59 into main Aug 5, 2026
1 check passed
@goulter
goulter deleted the GH-62 branch August 5, 2026 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants