ASVS icon indicating copy to clipboard operation
ASVS copied to clipboard

Published 20 hours ago • verified publisher icon OWASP

Adding OAuth 2.1 specifications

Open csfreak92 opened this issue 1 year ago • 3 comments

Update 0x51-V51-OAuth2.md based on recommendations from reviewers to reflect OAuth 2.1 specifications and comments from reviewers.

This Pull Request relates to issue #996 by taking into consideration the initial wave of recommendations.

csfreak92 avatar May 24 '24 04:05 csfreak92

Hi, I wrote my comment in https://github.com/OWASP/ASVS/issues/1965

In short, let's have an agreement on proposed requirement text before we do PR. I close this for now.

elarlang avatar May 24 '24 08:05 elarlang

Note, I reopened it at the moment. First I thought it was only about the one redirect_uri requirement, but there were more changes in.

elarlang avatar May 24 '24 21:05 elarlang

Hi Elar, noted.

csfreak92 avatar May 25 '24 06:05 csfreak92

So the requirement got into the repo and now there is opened some discussion over chapter texts (2 lines). I prefer we have a clear understanding of an issue before we go for PR and do not have discussion over them in the PR's.

elarlang avatar Aug 21 '24 15:08 elarlang

I close it now. Please see and follow: https://github.com/OWASP/ASVS/issues/996#issuecomment-2322847200

elarlang avatar Aug 31 '24 09:08 elarlang