Skip to content

Original user who imported the project is used to report commit statuses #6686

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

Closed
stsewd opened this issue Feb 20, 2020 · 1 comment · Fixed by #6839
Closed

Original user who imported the project is used to report commit statuses #6686

stsewd opened this issue Feb 20, 2020 · 1 comment · Fixed by #6839
Labels
Needed: design decision A core team decision is required

Comments

@stsewd
Copy link
Member

stsewd commented Feb 20, 2020

When a project has the PR builder feature enabled, they must check the webhook sends us the PR events and the Read the Docs auth app has permissions over the organization they are importing the project from.

The last thing is a problem when there are several members in the org, we use the credential from the user who first imported the project to report the commit status

build.project.remote_repository.users.first(),

This is also kind of related to #2759

@stsewd stsewd added the Needed: design decision A core team decision is required label Feb 20, 2020
@humitos
Copy link
Member

humitos commented Mar 6, 2020

We already talked about this and we mentioned two possible solutions:

  1. try all the maintainers of the project each time the status has to be reported
  2. try all the maintainers and cache the one that worked for next time first attempt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needed: design decision A core team decision is required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants