Skip to content
This repository was archived by the owner on Sep 6, 2025. It is now read-only.

PT-4024 Add option to use Zappr Github app's token for calls to Github#22

Merged
startnow65 merged 1 commit intomasterfrom
feature/use-zappr-apptoken
Jun 28, 2019
Merged

PT-4024 Add option to use Zappr Github app's token for calls to Github#22
startnow65 merged 1 commit intomasterfrom
feature/use-zappr-apptoken

Conversation

@startnow65
Copy link
Contributor

Background
API Calls to Zappr can be authorized by passing a Github token. With this approach, all call made by Zappr to Github would also use this token which implies that all github checks setup as part of this process would be tied to the owner of the Github token
Before now, we got the authentication cookie used by Zappr web (UI) to authenticate to the API and used that to also authenticate to Zappr API, this approach ensured that calls made by Zappr to Github uses Zappr's Github app token as expected.

Now
An endpoint has been added to Zappr to retrieve the internal Github token that is used to call Github as Zappr.
This PR uses that endpoint to retrieve this Github token and use that for API requests to Zappr

@antonu17
Copy link

antonu17 commented Jun 28, 2019

+1

Approved with Zappr Approved with Zappr Approved with Zappr Approved with Zappr

@startnow65 startnow65 requested a review from antonu17 June 28, 2019 11:26
@startnow65 startnow65 merged commit 2162f81 into master Jun 28, 2019
@startnow65 startnow65 deleted the feature/use-zappr-apptoken branch June 28, 2019 11:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants