search

ietf-wg-gnap / gnap-core-protocol

143 stars 26 forks source link

Add redirection status code security considerations #351

Closed pq2 closed 2 years ago

pq2 commented 2 years ago

Based on the OAuth 2.0 Security Best Current Practice

netlify[bot] commented 2 years ago

✔️ Deploy Preview for gnap-core-protocol-editors-draft ready!

🔨 Explore the source changes: f44214bf91761bd95f61433d60a4a2d0bf127f04

🔍 Inspect the deploy log: https://app.netlify.com/sites/gnap-core-protocol-editors-draft/deploys/61a942c2510ab50007db63ef

😎 Browse the preview: https://deploy-preview-351--gnap-core-protocol-editors-draft.netlify.app/draft-ietf-gnap-core-protocol

fimbault commented 2 years ago

Looks good, although there seems to be some whitespace issues and will need a new line in document history

pq2 commented 2 years ago

The changes to the whitespace are made automatically by any editor that respects the editorconfig file. I can undo them, but for anyone who wants to make changes in the future with an editor that respects editorconfig, those changes will then be made again automatically. Presumably these whitespace issues should be fixed by a separate commit in the main branch if the editorconfig file is to continue to be used.