Skip to content
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

endpoint URL in pop JWT? #38

Closed
tlodderstedt opened this issue Jul 30, 2023 · 4 comments
Closed

endpoint URL in pop JWT? #38

tlodderstedt opened this issue Jul 30, 2023 · 4 comments
Assignees

Comments

@tlodderstedt
Copy link
Contributor

Should the actual URL the client attestation pop JWT shall be sent to be included in the pop JWT?

DPoP has it but I assume the nonce is powerful enough to protect against replay but want to make sure we have thought about it.

@peppelinux
Copy link

I didn't get the "actual URL" , do you mean the htu value?

@tplooker
Copy link
Collaborator

tplooker commented Sep 6, 2023

@tlodderstedt can you confirm what you mean, I read your issue as meaning we should be including the endpoint that the PoP is going to be presented at (e.g the token endpoint)?

@tplooker tplooker self-assigned this Sep 6, 2023
@tlodderstedt
Copy link
Contributor Author

yes, that's what I mean. The PoP should be restricted to the endpoint URL it is sent to.

@tplooker
Copy link
Collaborator

The HTU claim in DPoP serves a couple of purposes which are freshness and audience constraint. With this draft the client attestation PoP already has normative text around populating the aud value for audience constraint and a mechanism around jti for a limited solution to detecting freshness (replay attacks). We also have #39 discussing a solution to a server generated nonce which will improve the freshness (replay attacks). With those mechanisms in place I dont believe we also need an HTU style claim.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants