n4o847 / seccamp-redos

A tool for detecting ReDoS vulnerabilities based on automata theory.

https://n4o847.github.io/seccamp-redos/

7 stars 4 forks source link

エラーハンドリングを実装 #46

Open yapatta opened 3 years ago

yapatta commented 3 years ago

解決する Issue

close #45

改善内容

buildChildを実行時にエラー発生した場合にエラーハンドリングしそれ相当のメッセージを表示(以前はハンドリングしておらず自動で落ちていた)

yapatta commented 3 years ago

急に思いついて作業しただけなので、思いついたときにレビューお願いしますw

n4o847 commented 3 years ago

あれ、レビューが消えてる

yapatta commented 3 years ago

レビューありがとうございます！(EitherとかResultみたいな雰囲気を醸し出したいなと思いました) エラーをcatchしないのは僕が誤解していたようです(test.tsと混同していた)。。。(detectReDoSのtry catchで拾ってくれていました) 三浦くんが言う通りtryは残すべきところは残したほうがよさそうですね〜後ほど修正します