helm / helm/pull-sizer

Check for operating repos doesn't normalize to lowercase

Offen
#7 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Go
Sterne
8
Forks
6
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Start in main.go around line 176, where the repository names are compared, and review the linked code path for the webhook check. Verify that mixed-case organization names work even when GITHUB_REPO_NAME uses different casing; done means the repository is accepted without the 403 error.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
55/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.