helm / helm/pull-sizer

Check for operating repos doesn't normalize to lowercase

オープン
#7 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
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

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

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