Skip to content

Improve logout message for asymmetric tokens#12587

Merged
bors merged 1 commit into
rust-lang:masterfrom
arlosi:paseto-logout
Aug 29, 2023
Merged

Improve logout message for asymmetric tokens#12587
bors merged 1 commit into
rust-lang:masterfrom
arlosi:paseto-logout

Conversation

@arlosi

@arlosi arlosi commented Aug 29, 2023

Copy link
Copy Markdown
Contributor

When doing cargo logout with an asymmetric token, it currently always succeeds with no message printed.

This changes the cargo:paseto provider to match the other providers and respond with NotFound if the user is not logged in. It also adds a message to indicate that the token was successfully removed (if it existed).

@rustbot

rustbot commented Aug 29, 2023

Copy link
Copy Markdown
Collaborator

r? @ehuss

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Aug 29, 2023
@epage

epage commented Aug 29, 2023

Copy link
Copy Markdown
Contributor

@bors r+

Thanks!

@bors

bors commented Aug 29, 2023

Copy link
Copy Markdown
Contributor

📌 Commit 0a5050c has been approved by epage

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 29, 2023
@bors

bors commented Aug 29, 2023

Copy link
Copy Markdown
Contributor

⌛ Testing commit 0a5050c with merge e597acd...

@bors

bors commented Aug 29, 2023

Copy link
Copy Markdown
Contributor

☀️ Test successful - checks-actions
Approved by: epage
Pushing e597acd to master...

@bors bors merged commit e597acd into rust-lang:master Aug 29, 2023
bors added a commit to rust-lang-ci/rust that referenced this pull request Aug 29, 2023
Update cargo

18 commits in 925280f028db3a322935e040719a0754703947cf..96fe1c9e1aecd8f57063e3753969bb6418fd2fd5
2023-08-25 21:16:44 +0000 to 2023-08-29 20:10:34 +0000
- fix(lints): Fail when overriding inherited lints (rust-lang/cargo#12584)
- cargo install: suggest --git when package name is url (rust-lang/cargo#12575)
- chore: remove unstable-options for logout (rust-lang/cargo#12588)
- Improve logout message for asymmetric tokens (rust-lang/cargo#12587)
- fix(update): Remove references to -p in help (rust-lang/cargo#12586)
- fix(update): Make `-p` more convenient by being positional (rust-lang/cargo#12545)
- Set tracing target for networking messages. (rust-lang/cargo#12582)
- Retry docs (rust-lang/cargo#12583)
- feat(resolver): **Very** preliminary MSRV resolver support (rust-lang/cargo#12560)
- Update git2 (rust-lang/cargo#12580)
- Explain how `version` works for `git` dependencies (rust-lang/cargo#12270)
- Improve deserialization errors of untagged enums (rust-lang/cargo#12574)
- Add support for `target.'cfg(..)'.linker` (rust-lang/cargo#12535)
- Improve resolver version mismatch warning (rust-lang/cargo#12573)
- Stabilize `--keep-going` (rust-lang/cargo#12568)
- Define {{command}} for use in src/doc/man/includes (rust-lang/cargo#12570)
- Update serde (rust-lang/cargo#12569)
- chore: add missing `windows-sys` features back (rust-lang/cargo#12563)

r? ghost
@ehuss ehuss added this to the 1.74.0 milestone Sep 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants