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

assign CVE ID for open redirect fixed in 1.50.0 #45

Closed
zmanion opened this issue Aug 15, 2022 · 4 comments
Closed

assign CVE ID for open redirect fixed in 1.50.0 #45

zmanion opened this issue Aug 15, 2022 · 4 comments
Assignees
Labels
security Affects security

Comments

@zmanion
Copy link
Contributor

zmanion commented Aug 15, 2022

@JLLeitschuh (with help from CodeQL) reported this open redirect, which was fixed in 1.50.0:

https://www.kb.cert.org/vince/comm/login/?next=https%3A%2F%2Fexample.com

if is_safe_url(next_url,set(settings.ALLOWED_HOSTS),True):

b986a86

While not a serious issue:
https://bughunters.google.com/learn/invalid-reports/web-platform/navigation/6680364896223232/open-redirectors

...it could be considered CVE-worthy and handled as such. I'd be happy to write it up if that helps.

@JLLeitschuh
Copy link

JLLeitschuh commented Aug 15, 2022

It would be good if a CVE number was assigned

@sei-vsarvepalli sei-vsarvepalli self-assigned this Aug 15, 2022
@sei-vsarvepalli sei-vsarvepalli added the security Affects security label Aug 15, 2022
@sei-vsarvepalli
Copy link
Contributor

Hello @JLLeitschuh

Mea culpa - Sorry for the delay. I promised to get a CVE and also give you credit. Hopefully next week will be less crazy and will get the CVE and update this issue.

Thanks
Vijay

@sei-vsarvepalli
Copy link
Contributor

Hello @zmanion and @JLLeitschuh

Reserved and published this with CVE-2022-25799 with credit to @JLLeitschuh

Thanks
Vijay

@zmanion
Copy link
Contributor Author

zmanion commented Aug 17, 2022

Thanks @sei-vsarvepalli @JLLeitschuh and @attritionorg. #dogfood!

@zmanion zmanion closed this as completed Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security Affects security
Projects
None yet
Development

No branches or pull requests

3 participants