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の脆弱性 #89

Open
prograti opened this issue Jan 13, 2020 · 3 comments
Open

XSSの脆弱性 #89

prograti opened this issue Jan 13, 2020 · 3 comments
Labels
bug Something isn't working

Comments

@prograti
Copy link

外部APIを叩いて取得したtoot(JSON)をパースしてそのまま表示していますが、エスケープ処理がされていないため悪意のあるJSONデータが返却された場合に任意のjavascriptを実行することが可能です。

Qiitadon以外のインスタンスも指定できる仕様ですので、外部データは常に危険であるという前提でエスケープ処理を行う必要があります。

xss payload : https://api.myjson.com/bins/158yga

@yume-yu yume-yu added the bug Something isn't working label Jan 13, 2020
@KEINOS
Copy link
Member

KEINOS commented Jan 15, 2020

#91 の Tabnabbing も勉強になったから対応してみようかな。(できるとは言ってない

@KEINOS
Copy link
Member

KEINOS commented Jan 15, 2020

PR #104 で対応しています。無理だったらごめんちゃい

@yumetodo
Copy link
Contributor

#104 (comment)
にも書きましたが、検証例おいておきますね
https://deploy-preview-104--zen-edison-40804e.netlify.com/p.html?i=https://repos.irregular-at-tus.work&t=1
https://repos.irregular-at-tus.work に検証のためにファイルおきたい場合は声かけてください。

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

No branches or pull requests

4 participants