Dead check in UpdateAccessTokenFilter.java, possible security issue
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- java
調査の方向性
src/main/java/org/ohdsi/webapi/shiro/filters/UpdateAccessTokenFilter.java の124行目から始め、jwt変数がどのように代入され、周辺のフィルターフローでどのように使用されているかを追跡します。チェックの本来の意図を判断し、適切な修正を文書化または実装します。完了条件は、jwtの処理が説明のないデッドチェックではなくなるか、有効な目的がない場合に削除されることです。
索引モデルが issue の本文から書いたものです。
説明
Expected behavior
The jwt variable should not always be null or the check should be removed.
Actual behavior
The jwt is always null here:
https://github.com/OHDSI/WebAPI/blob/723eb4cbe470dc58d1fa340add5c2d9cc89a4e28/src/main/java/org/ohdsi/webapi/shiro/filters/UpdateAccessTokenFilter.java#L124
Steps to reproduce behavior
Just reading the code is sufficient to conclude that.
I am currently making changes to this class, so it would be very useful to know what the original intention of this part was.
- 主要言語
- Java
- スター
- 153
- フォーク
- 187
- 平均マージ
- 1日 1時間
- マージ済み PR（30日）
- 15
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
OHDSI/WebAPI のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
CDM cache warming still fails when cache.achilles.usePersonCount=false (missing `cc` alias in getBatchConceptRecordCount.sql)対応中かも @DG47 が 4 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
GET /conceptset/{id}/expression returns items in nondeterministic order (query has no ORDER BY)オープン
難易度 3/5 1〜2日 初心者へのやさしさ 67/100
OHDSI/WebAPI#2543 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
OHDSI/WebAPI#2513 · コメント 3 件 ·
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
ome/ome-codecs#48 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
tonihele/OpenKeeper#672 ·
メンテナーはふだん 4 日以内に返信
-
Shaded google-api-client trust stores still include AAA Certificate Services (eMudhra CA compromise)オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
snowflakedb/snowflake-jdbc#2763 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 79/100
java-native-access/jna#1746 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
ControlSystemStudio/phoebus#3983 ·
メンテナーはふだん 2 日以内に返信