Repository navigation
feat!: Add auth-logout input and run pixi auth status after login - #291
Merged
Merged
Conversation
pavelzw
changed the title
Replace persist-credentials with auth-logout (post | after-install | never). The default (post) logs out in the post step at the end of the job. Closes #290
pavelzw
changed the title
2 of 4 tasks
Collaborator
Author
|
fyi @AndreasAlbertQC |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #290
Changes
pixi auth statusafter logging in, in the same "Logging in to private channel" log group.persist-credentialsinput with anauth-logoutenum that controls whenpixi auth logout <auth-host>runs:post(default, new): credentials stay available to all subsequent steps and are removed in the post step at the end of the jobafter-install: log out at the end of the action (previouslypersist-credentials: false)never: never log out (previouslypersist-credentials: true, the old default)persist-credentialsnow fails with an error that points toauth-logout, instead of being silently ignored.Migration
persist-credentials: falseauth-logout: after-installpersist-credentials: true/ not setauth-logout: neverto keep the old behavior, or leave it unset to log out in the post stepIf updating documentation: