-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[GH Request] Investigate permissions of Github token being used to fetch data from the openedx org #863
Comments
Thank you for your report! @openedx/axim-oncall will triage within a business day. Simple requests usually take 2-3 business days to resolve; more complex requests could take longer. |
Will move to blocked until Arib-BOM chimes in with further details. |
During the investigation of why the dependabot_alerts checks have inconsistent data, found that data related to dependabot_alerts for all openedx repos is missing. On further digging, found out that the token has limited access to fetch that data from openedx repos. Getting the following error for all openedx repositories.
|
There's more here than simply updating permissions. If I understand correctly, you are trying to connect from a workflow that is running in the edx GitHub organization to an endpoint providing data about repositories in the openedx GitHub organization. If I understand correctly, we'll need to take an expanded approach. Please confirm that I've understood correctly. |
Coming from the same conversation with Ed about this: In #738 (comment) Kyle mentioned that the permissions should be correct for the |
I was thinking of something similar to what you suggest @jmbowman . I think that running the workflow on the openedx side and giving your script access to the repository where the data output is stored would align with GH permissions models quite cleanly. It also has the added advantage of making the openedx data available in a consistent format to other community members. |
Given there's been no follow up here, I'm going to close. Please reopen if there's more to do. |
Firm Name
2U (edx)
Urgency
Low (2 weeks)
Problem/Request
A follow up from #738
After the initial work from Axim and Arch Bom, Jeremy proposed follow up work here: #530 (comment)
I'm told that Arbi-BOM also plans to reach out to help investigate. This is just me getting the ball rolling for them since this intersects with my work on SWG.
Based on github docs, since these are public repos, confirm that the token used for the job has public_repo scope
https://docs.github.com/en/free-pro-team@latest/rest/dependabot/alerts?apiVersion=2022-11-28#list-dependabot-alerts-for-a-repository
Reasoning
We're trying to get dependabot alert data onto the repo health data for open edx repos.
The text was updated successfully, but these errors were encountered: