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

[VULNERABILITY] There is a VULNERABILITY on depending package url-regex. #6

Closed
isnot opened this issue Jul 18, 2020 · 2 comments · Fixed by #7
Closed

[VULNERABILITY] There is a VULNERABILITY on depending package url-regex. #6

isnot opened this issue Jul 18, 2020 · 2 comments · Fixed by #7
Assignees
Labels
bug Something isn't working

Comments

@isnot
Copy link
Owner

isnot commented Jul 18, 2020

disclosure

description (ja)

私が公開している当プロダクトに、脆弱性が報告されていることに気づきました。
差し当たり、本件は既にパブリックな場所で公開されている状態のため、私もここで、公開することとします。

脆弱性の内容は、正規表現を利用したDoS攻撃に対する問題(ReDoS)です。

問題の原因箇所は、依存ライブラリの中にあります。

  • hexo-tag-google-photos-album
    • metascraper
      • metascraper-helper
        • url-regex

一方で、私が作った部分では、問題を引き起こす可能性のある方法で、上記問題の見つかった機能を利用していることを確認しました。 /index.js#L81

solution 対応方針

2020/07/18 時点で、まだ決定していません。
候補としては

  • metascraperに渡す前に、パラメーター(url)を独自にチェックする
  • metascraper を使わない
  • url-regexが修正され、その修正されたバージョンを使ったmetascraper-helperが公開されるのを待つ

それぞれメリット/デメリットあります。

引き続き対応予定です。

@isnot isnot added the bug Something isn't working label Jul 18, 2020
@isnot isnot self-assigned this Jul 18, 2020
@isnot
Copy link
Owner Author

isnot commented Jul 18, 2020

依存ライブラリ側での対応状況をまとめます。

私の個人的理解では、今の所、以下の感じです。

url-regex => 1年以上、コミットがない。開発が停滞もしくは中止されている可能性がある。issueでは議論が続いているが、メイン開発者は、本件に過去80日間反応していない。

metascraper-helper => 24日前に、issueが立った。メイン開発者(と思われる)は、次の理由で即座にClose。内容は「url-regexの問題なので、そちらで議論してほしい」

@isnot isnot mentioned this issue Jul 29, 2020
@isnot isnot closed this as completed in #7 Jul 29, 2020
@isnot
Copy link
Owner Author

isnot commented Jul 29, 2020

solution 対応方針 -> metascraper を使わない
で対処しました。at #7

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.

1 participant