github / github/codeql

LGTM.com - false positive - XXX may be at an arbitrary position in the sanitized URL.

オープン
#4,817 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
false-positive Python
主要言語
CodeQL
スター
10.1k
フォーク
2.1k
平均マージ
2日 15時間
マージ済み PR(30日)
141

説明

**Description of the false positive**

We got a warning that a string may be at an arbitrary position in the sanitized URL. After looking at the code and at the warning's explanation I can definitely understand the sentiment of the warning but it is a bit opinionated. Can't assume that every time there is such a check there is a vulnerability hidden.

For example in this code's case it checks if `infura.io` is in the URL (can have various permutations so not using `startswith`) and if it is we try to make it easier for the user by using a different querying range. There is no danger whatsoever if somehow there is infura.io in a non correct part of the url.

**URL to the alert on the project page on LGTM.com**

https://lgtm.com/projects/g/rotki/rotki/snapshot/67b7fedfc510b557a4ecd6d6d8c5bd32dbfeb2e6/files/rotkehlchen/chain/ethereum/manager.py?sort=name&dir=ASC&mode=heatmap#x11dc760fe6c8da89:1

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。