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

Post-CAPTCHA "Continue browsing" link is not useful #3126

Open
esabol opened this issue Jul 2, 2024 · 5 comments
Open

Post-CAPTCHA "Continue browsing" link is not useful #3126

esabol opened this issue Jul 2, 2024 · 5 comments
Labels

Comments

@esabol
Copy link

esabol commented Jul 2, 2024

After completing the CAPTCHA, you get a "Success" message. Underneath that, there's a link to "Continue browsing". I would expect this link to take me to back to the web page of the module where I clicked the "++" button. It doesn't. It takes me to another CAPTCHA instead. Not very useful.

@oalders
Copy link
Member

oalders commented Jul 2, 2024

If you have a look at the HTML is there actually a target URL in the "Continue browsing" link?

@esabol
Copy link
Author

esabol commented Jul 2, 2024

If you have a look at the HTML is there actually a target URL in the "Continue browsing" link?

Good question, but I'm not sure how to get the website to ask me to enter the CAPTCHA again. I can't seem to reproduce it right now.

@oalders
Copy link
Member

oalders commented Jul 3, 2024

You'd probably need a new session to hit that requirement. So, either logging out or an incognito window might be enough.

@esabol
Copy link
Author

esabol commented Jul 3, 2024

Yeah, that's what I thought, and I tried it an incognito window yesterday. It prompted me to login when I clicked on the "++" button, but it didn't give me a CAPTCHA after I logged in.

I found the "Continue browsing" HTML in the repository. It links to a variable, $referer, but I don't see where that's defined. But I think what's really needed is the page before $referer since $referer is the CAPTCHA page.

<a href="[% $referer %]">Continue browsing</a>

@oalders
Copy link
Member

oalders commented Jul 4, 2024

I believe it's supposed to be set via https://github.com/metacpan/metacpan-web/blob/master/lib/MetaCPAN/Web/Controller/Account/Turing.pm#L20

It's possible that this parameter is not being included in the POST.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants