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

XSS Due to insufficient checks on the external resources #1026

Closed
kahla-sec opened this issue Feb 5, 2023 · 3 comments · Fixed by #1069
Closed

XSS Due to insufficient checks on the external resources #1026

kahla-sec opened this issue Feb 5, 2023 · 3 comments · Fixed by #1069
Labels
bug Something isn't working

Comments

@kahla-sec
Copy link

kahla-sec commented Feb 5, 2023

Describe the bug

The memos application allows users to add resources from an external link. It was found that there are no checks on the added link, thus allowing a malicious user to introduce links starting with javascript: scheme . If the user creates a public note with an external resource pointing to a malicious link javascript:alert(1), it would be possible to achieve XSS and steal the other users private notes or impersonate their accounts .
It's advised to only allow external resources that are served from https:// websites in order to mitigate this vulnerability.

Steps to reproduce

1- Create a resource and choose External Link
2- Type the following link: javascript:alert(document.domain)
3- Choose the type as application/* and add the resource
4- Now create a new public note and add the malicious external resource
5- If a user or the admin clicks the external resource, the malicious JS code will be executed on their behalf

Screenshots or additional context

No response

@kahla-sec kahla-sec added the bug Something isn't working label Feb 5, 2023
@kahla-sec
Copy link
Author

Any answer from the maintainers ?
@boojack Maybe you can help regarding this :)

@boojack
Copy link
Collaborator

boojack commented Feb 8, 2023

@kahla-sec Thanks so much. We will fix this as soon as possible.

@kahla-sec
Copy link
Author

Thanks a lot! I'll be glad to retest it when it's done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants