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

Create an account just for token creation #140

Open
maelle opened this issue May 2, 2022 · 6 comments
Open

Create an account just for token creation #140

maelle opened this issue May 2, 2022 · 6 comments

Comments

@maelle
Copy link
Contributor

maelle commented May 2, 2022

as mentioned by @assignUser #123 (comment)

Advantages

  • no access to other repos
  • if using an address like [email protected] and well named tokens, several people would be notified of tokens needing to be renewed.
@mpadge
Copy link
Member

mpadge commented Apr 18, 2023

This gained a boost in importance this week, with the discovery that GitHub must have somehow changed their token handling. Up until now, the main token shared between pkgcheck and pkgcheck-action (called "RRT_TOKEN") was a simple repo-level token that could be created by anyone with admin access, yet did not need to be a personal token. That recently stopped working, and the use of the token here:

- name: Trigger pkgcheck-action build
if: ${{ github.event_name != 'pull_request' }}
uses: actions/github-script@v6
with:
github-token: ${{ secrets.RRT_TOKEN }}

now only works with a personal token copied over to the equivalent token values here and in pkgcheck-action. I'll update the docs straight away to indicate this. The process will then definitely be easier and more robust if we had an account for token creation.

@maelle
Copy link
Contributor Author

maelle commented Apr 18, 2023

why not use ropenscibot then?

@mpadge
Copy link
Member

mpadge commented Apr 18, 2023

Yep, my thought too. We would then need to ensure that at least one person with admin there was also watching notifications from pkgcheck.

@maelle
Copy link
Contributor Author

maelle commented Apr 18, 2023

what sort of notifications would be relevant? the token expiration?

@mpadge
Copy link
Member

mpadge commented Apr 18, 2023

Yes. Technically they'd just have to watch whatever issue was given here:

issue_number: 123,

which is where the token expiration notifications come. Shall we set that up then?

@mpadge
Copy link
Member

mpadge commented Apr 18, 2023

Description of tokens added in this ropensci-review-tools commit, including a section on "Assigning 'RRT_TOKEN' updates to a different person." Maybe that's actually sufficient for now? What do you think @maelle @assignUser

Rendered section on tokens is here, with description of maintance-handover here.

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