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

Login & authentication rework, part 2: prompt cleanup #3392

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

apostasie
Copy link
Contributor

This is a small PR that comes on top of #3293 and should be merged after it, in preparation for more login cleanup.

Its purpose is solely to isolate user input prompting cleanly, and provide proper error reporting for it.

This is purely refactoring and cleanup.

@apostasie apostasie changed the title Login & authentication rework, part 2: prompt cleanup [WIP] Login & authentication rework, part 2: prompt cleanup Sep 1, 2024
@apostasie apostasie marked this pull request as ready for review September 1, 2024 23:43
@apostasie apostasie changed the title [WIP] Login & authentication rework, part 2: prompt cleanup Login & authentication rework, part 2: prompt cleanup Sep 1, 2024
@apostasie
Copy link
Contributor Author

Will rebase once Part 1 lands.

Signed-off-by: apostasie <spam_blackhole@farcloser.world>
@apostasie
Copy link
Contributor Author

Rebased.
CI is green.

@AkihiroSuda AkihiroSuda added this to the v2.0.0 milestone Sep 4, 2024
@AkihiroSuda AkihiroSuda added the area/login authentification/ login label Sep 4, 2024
@AkihiroSuda AkihiroSuda merged commit 53d898d into containerd:main Sep 4, 2024
22 checks passed
@AkihiroSuda
Copy link
Member

Thanks

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

Successfully merging this pull request may close these issues.

2 participants