Skip to content
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

Use deployer's oauth token for GitHub deployments API #3464

Open
ibrahima opened this issue Jul 30, 2019 · 3 comments
Open

Use deployer's oauth token for GitHub deployments API #3464

ibrahima opened this issue Jul 30, 2019 · 3 comments

Comments

@ibrahima
Copy link
Contributor

I realize this is probably not possible right now because of #51, but it would be nice if the GitHub create deployment API would show up with the actual deployer having deployed instead of as the "service user" that was configured in the .env file. It seems like the deployments API doesn't let you say who deployed via the API, so it will always show up as the user for Samson. Not super important since we can look up that info by looking at the deploy on Samson, but more of a nice to have. Just bringing this up because I'm not sure if this is already possible somehow and I've misconfigured something.

@grosser
Copy link
Contributor

grosser commented Aug 1, 2019

I don't think that works since we'd have to auth as that user ... unless saying who deployed is part of the deployment api.

@ibrahima
Copy link
Contributor Author

Yeah I guess I'm requesting that you could auth as the user, I think that's how the deployment API is meant to find out who deployed (otherwise, it's just something like "Samson deployed blah..."). But I don't know if you get/store user's OAuth tokens right now.

It would be nice if the API just let you specify the username who initiated the deploy but it seems like it does not let you do that: https://developer.github.com/v3/repos/deployments/#create-a-deployment. Maybe I should make a feature request to the GitHub API team.

@grosser
Copy link
Contributor

grosser commented Dec 20, 2019

Sending a username over would be much simpler :)
... I doubt the oauth token samson gets during login works with anything but samson.
So only other way would be a per-user column of oauth token that we then send ... which would be hard to configure for each user ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants