helm / helm/pull-sizer

Check for operating repos doesn't normalize to lowercase

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Go
Stars
8
Forks
6
PR merge metrics
No merged PRs in 30d

Description

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

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.