-
Notifications
You must be signed in to change notification settings - Fork 68
Closed
Labels
area/githubGitHub-forge relatedGitHub-forge relatedgood-first-issueGood for newcomersGood for newcomerskind/bugAn unexpected problem or behaviorAn unexpected problem or behavior
Description
Discovered in: #759
Cause: An exception is raised when querying the users with permissions on a project, if the current user accessing the API doesn't have any permission themselves. However the current implementation checks generic GitHub exception which can lead to catching also other kind of errors, e.g. 50x
ogr/ogr/services/github/project.py
Lines 210 to 214 in 6e3e246
| except github.GithubException: | |
| logger.debug( | |
| "Current Github token must have push access to view repository permissions." | |
| ) | |
| return set() |
Metadata
Metadata
Assignees
Labels
area/githubGitHub-forge relatedGitHub-forge relatedgood-first-issueGood for newcomersGood for newcomerskind/bugAn unexpected problem or behaviorAn unexpected problem or behavior
Type
Projects
Status
done