Skip to content

Fix csrf - #998

Merged
TG1999 merged 4 commits into
mainfrom
fix-csrf
Nov 9, 2022
Merged

Fix csrf#998
TG1999 merged 4 commits into
mainfrom
fix-csrf

Conversation

@pombredanne

Copy link
Copy Markdown
Member

This PR fixes API key request issues and improves the email sent.

Signed-off-by: Philippe Ombredanne pombredanne@nexb.com

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
This is needed for effective CSRF proptection on deployments

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
* Enhance the email message content.
* Streamline About content in web UI.
* Format code and fix typos.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
pombredanne requested a review from TG1999 November 9, 2022 10:53

@TG1999 TG1999 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@TG1999
TG1999 merged commit 0a0460b into main Nov 9, 2022
@TG1999
TG1999 deleted the fix-csrf branch November 9, 2022 10:55
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.

2 participants