Skip to content

Comments

fix(oauth2): escape authorize form values in login HTML#18

Open
unvalley wants to merge 1 commit intomainfrom
fix/oauth-login-html-escaping
Open

fix(oauth2): escape authorize form values in login HTML#18
unvalley wants to merge 1 commit intomainfrom
fix/oauth-login-html-escaping

Conversation

@unvalley
Copy link
Owner

Summary

  • HTML-escape authorize query values before embedding them in login form hidden fields
  • prevent reflected script or attribute injection via state, nonce, redirect_uri, and related fields
  • add a unit test for escaped output

Testing

  • cargo test test_generate_login_html_escapes_attribute_values
  • cargo test test_authorization_code_flow

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant