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

Add advisory for will_paginate #272

Open
aripollak opened this issue Oct 28, 2016 · 3 comments
Open

Add advisory for will_paginate #272

aripollak opened this issue Oct 28, 2016 · 3 comments

Comments

@aripollak
Copy link

aripollak commented Oct 28, 2016

Apparently there was a security issue fixed here: mislav/will_paginate@ec9b985 and here: mislav/will_paginate@ab55687

It looks like versions before 3.1.2, 3.0.9, and are vulnerable, but this has no CVE and I'm not sure exactly what the vulnerability was.

@aripollak
Copy link
Author

With a bit of experimentation, I discovered that adding &script_name=https://www.example.com to a page with will_paginate links would result in the links being rewritten to be https://www.example.com... instead of the intended site. I guess someone could use this in a social engineering attack by sending someone a link with &script_name being a malicious site and hoping they click on the pagination links and getting fooled, but I'm not sure if it's any worse than that.

@jasnow
Copy link
Contributor

jasnow commented Jun 3, 2023

@aripollak or @phillmv - Is this issue related to this existing advisory - gems/will_paginate/CVE-2013-6459.yml ?
Thanks

@aripollak
Copy link
Author

@jasnow I think that's a different issue.

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

No branches or pull requests

3 participants