-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat: allow non-admin config tokens to rotate using rotation API #90
Conversation
I updated the test data to support the |
The " |
@ilijamt I don't mean to pressure you (much), but I need to know if you could review/merge/release this, or if I need to try to figure out how to get it released in my fork. Thanks in advance :) |
I'm checking it, I'm running into some issues. Need to do some more testing. And checking to see what are the possibilities, and if there is another way. |
Let me know, happy to try to help. The " |
It has been more than a month, I know this isn't probably your primary job, but can I do anything to move this along, please and thanks in advance? :) |
I used part of your code in #97 thanks for the help. I decided instead of keeping two paths for token rotation to just use it's own endpoint instead, so that will make it easier in the future. |
Initial attempt at catching non-admin users and using the token rotation API
Rotation tested!
I was going to add acceptance testing, but it doesn't look like that is very far along yet. ;)
Closes #89