Check for operating repos doesn't normalize to lowercase
- 主要言語
- Go
- スター
- 8
- フォーク
- 6
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
I just installed this for a repo with an organization whose name contains mixed case. I did not think to mix the case when I provided the `GITHUB_REPO_NAME` environment variable. This cased a 403 error with an error of `{"message":"Not configured for this repository"}`. I updated the organization's name in the environment variable to match the case as found in the payload and then the webhook succeeded.
Recommend normalizing both `repoSplit[0]` and `checkSplit[0]` to lowercase for comparison.
https://github.com/helm/pull-sizer/blob/bb557b11dfb698c211d6a9dd70935c93a97642ee/main.go#L176
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
調査の方向性
main.go の 176 行目付近にある、リポジトリ名を比較している箇所から始め、webhook のチェックに関連するコードパスを確認してください。GITHUB_REPO_NAME で大文字と小文字の使い方が異なる場合でも、大文字と小文字が混在する組織名が機能することを確認してください。403 エラーなしでリポジトリが受け入れられれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- go
- 領域
- backend
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 55/100