Skip to content

Conversation

di
Copy link
Member

@di di commented Mar 24, 2023

This makes our reCAPTCHA more likely to work for a global user base, per https://developers.google.com/recaptcha/docs/faq#can-i-use-recaptcha-globally

@di di requested a review from a team as a code owner March 24, 2023 14:46
@ewdurbin
Copy link
Member

We probably need CSP updates for this, no?

@di
Copy link
Member Author

di commented Mar 24, 2023

We probably need CSP updates for this, no?

Yeah, that's the changes to warehouse/recaptcha.py

@ewdurbin
Copy link
Member

ah, sorry I expected that to be in the csp.py file. I understand now :)

@ewdurbin ewdurbin merged commit 9458ec9 into pypi:main Mar 24, 2023
@di di deleted the use-recaptcha.net branch March 24, 2023 15:03
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