Remove note on IdP to validate nonce (#582)

fedidcg / FedCM

A privacy preserving identity exchange Web API

https://fedidcg.github.io/FedCM

Other

357 stars 66 forks source link

Remove note on IdP to validate nonce (#582) #583

Closed obfuscoder closed 1 month ago

obfuscoder commented 1 month ago

Preview | Diff

cbiesinger commented 1 month ago

lgtm but could you resolve the IPR issue as described in https://labs.w3.org/repo-manager/pr/id/fedidcg/FedCM/583 ?

samuelgoto commented 1 month ago

LGTM++

npm1 commented 1 month ago

Yea ideally you can fix the affiliation. But I'm also open to consider this a non-substantive change since it just removes a note

obfuscoder commented 1 month ago

As this PR is just about removing content, you could probably waive this requirement for this one. I'll try to get this IPR sorted out with my company, but it willl probably take a while.

samuelgoto commented 1 month ago

SGTM.

Waving as non-substantive.