alibaba/Sentinel

About AuthorityRuleChecker.The PassCheck() method implementation problem

Open

#1,753 opened on 2020年9月23日

GitHub で見る
 (3 comments) (0 reactions) (0 assignees)Java (8,150 forks)batch import
good first issue

Repository metrics

Stars
 (23,109 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

Type: bug report

Describe what happened (or what feature you want)

This class is primarily used to determine the match of the context rules corresponding to the resource.Why do I need to repeat the validation with indexOf() and EQUALS methods?

Anything else we need to know?

com.alibaba.csp.sentinel.slots.block.authority.AuthorityRuleChecker

コントリビューターガイド

調査方針
AuthorityRuleChecker.PassCheck()メソッドの実装を調べ、ルールマッチングでindexOf()とequals()が重複して使用されている理由を理解します。バグか意図的かを判断し、冗長なチェックを削除することを提案します。
技術スタック
java
領域
backendapi
Issue 種別
バグ
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3時間
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
アクティブ
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
JavaSentinel basics
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
40