Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ReCAPTCHA proxy is misconfigured #7174

Closed
ctgraham opened this issue Jul 7, 2021 · 2 comments
Closed

ReCAPTCHA proxy is misconfigured #7174

ctgraham opened this issue Jul 7, 2021 · 2 comments
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Milestone

Comments

@ctgraham
Copy link
Collaborator

ctgraham commented Jul 7, 2021

Describe the bug
3.3 changes the config.inc.php proxy settings and Guzzle, without touching ReCAPTCHA's usage.

To Reproduce
Steps to reproduce the behavior:

  1. Configure 3.3 with new proxy settings
  2. Enable and attempt ReCAPTCHA
  3. ReCAPTCHA uses old (non-existent) proxy settings

https://forum.pkp.sfu.ca/t/you-did-not-pass-the-validation-check-used-to-prevent-spam-submissions/48023/34?u=ctgraham

What application are you using?
OJS 3.3.0-6

Additional information
#6539 updates ReCAPTCHA to use new proxy settings (courtesy of Guzzle) in 3.4

@NateWr NateWr added the Bug:1:Low A bug that does not have a severe consequence or affects a small number of users. label Jul 12, 2021
@NateWr NateWr added this to the OJS/OMP/OPS 3.3.0-8 milestone Jul 12, 2021
@NateWr
Copy link
Member

NateWr commented Jul 12, 2021

cc @jonasraoni and @asmecher It looks like part of #6539 needs to be back ported for 3.3.

@asmecher
Copy link
Member

This should be resolved by #7226, which tests fine locally (without proxy) -- but I'd love confirmation that it resolves the issue in a proxy scenario too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug:1:Low A bug that does not have a severe consequence or affects a small number of users.
Projects
None yet
Development

No branches or pull requests

3 participants