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.

Sign up for GitHub

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

Jump to bottom

Show if an authkey has been used already in the CLI #157

Closed
wants to merge 1 commit into from
Closed

Show if an authkey has been used already in the CLI #157

wants to merge 1 commit into from

Conversation

juanfont
Copy link
Owner

This PR implements #154 by adding a new AlreadyUsed field in PreAuthKey - but does not store it in the DB. Instead, it is filled when fetching the auth keys from the DB.

@juanfont juanfont changed the title Show if a authkey has been used already in the CLI Show if an authkey has been used already in the CLI Oct 12, 2021
@juanfont juanfont requested review from cure and kradalby October 12, 2021 22:49
@juanfont
Copy link
Owner Author

Going to close this PR, as does not solve #158.

@juanfont juanfont closed this Oct 13, 2021
juanfont added a commit that referenced this pull request Oct 13, 2021
@juanfont
Add AlreadyUsed field to Auth Keys (fixes #157 and #158)
30788e1
@juanfont juanfont mentioned this pull request Oct 13, 2021
Merged
@juanfont juanfont deleted the show-pak-already-used branch October 16, 2021 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cure cure Awaiting requested review from cure

@kradalby kradalby Awaiting requested review from kradalby kradalby is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@juanfont