-
Notifications
You must be signed in to change notification settings - Fork 231
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
[CVE-2021-26722] Reflected Cross-Site Scripting in search bar. #341
Comments
Hi @renniepak, thanks for bringing this to our attention. Unfortunately, we weren't previously informed of the issue by the Information Security Response Center but now that we are aware of this problem we will be addressing it with the highest priority. |
Hi @diegocepedaw, Thanks for the quick response. If there are any questions, I'm happy to help. |
Opened PR #342 to address the issue and will be requesting a further security review from our security team to make sure there are no other similar issues. |
Hi!
I've found a reflected cross-site scripting vulnerability in Oncall's search bar. I've reported this issue to the LinkedIn Information Security Response Center back in September 2020 but Oncall still seems vulnerable to this date. Therefore I decided to report it here.
Reproduction
http://[OnCallHost]/query/%3Cimg%20src=x%20onerror=alert(document.domain)%3E/all
<img src=x onerror=alert(document.domain)>
Result
By clicking the search bar, a search will be done to the search API endpoint. Because nothing can be found a
No results found for "<img src="x" onerror="alert(document.domain)">"
message will be shown. Because this message includes the search query and lacks the proper HTML output encoding, the query is interpreted as HTML/JS and an alert containing thedocument.domain
is shown.Impact
If an attacker can control a script that is executed in the victim's browser, then they can typically fully compromise that user. Amongst other things, the attacker can:
Mitigation
In general, effectively preventing XSS vulnerabilities is likely to involve a combination of the following measures:
The text was updated successfully, but these errors were encountered: