Skip to content

[Fix-17995][API] Non-admin users cannot delete their own access tokens#17997

Merged
SbloodyS merged 2 commits intoapache:devfrom
dill21yu:fix-delete-access-tokens
Feb 27, 2026
Merged

[Fix-17995][API] Non-admin users cannot delete their own access tokens#17997
SbloodyS merged 2 commits intoapache:devfrom
dill21yu:fix-delete-access-tokens

Conversation

@dill21yu
Copy link
Contributor

Was this PR generated or assisted by AI?

NO

Purpose of the pull request

to close #17995

Verify this pull request

Pull Request Notice

Pull Request Notice

If your pull request contains incompatible change, you should also add it to docs/docs/en/guide/upgrade/incompatible.md

@SbloodyS SbloodyS added the bug Something isn't working label Feb 25, 2026
@SbloodyS SbloodyS added this to the 3.4.1 milestone Feb 25, 2026
@SbloodyS SbloodyS changed the title [Fix-17995][API]Non-admin users cannot delete their own access tokens [Fix-17995][API] Non-admin users cannot delete their own access tokens Feb 25, 2026
ruanwenjun
ruanwenjun previously approved these changes Feb 25, 2026
Copy link
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

SbloodyS
SbloodyS previously approved these changes Feb 26, 2026
Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@SbloodyS
Copy link
Member

Please fix the failed UT @dill21yu

@dill21yu dill21yu dismissed stale reviews from SbloodyS and ruanwenjun via 09693a6 February 27, 2026 03:11
@github-actions github-actions bot added the test label Feb 27, 2026
@ruanwenjun ruanwenjun force-pushed the fix-delete-access-tokens branch from 62b6d67 to 17dddf5 Compare February 27, 2026 05:27
@sonarqubecloud
Copy link

Copy link
Member

@ruanwenjun ruanwenjun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@SbloodyS SbloodyS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@SbloodyS SbloodyS merged commit cfca07e into apache:dev Feb 27, 2026
75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend bug Something isn't working test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug][API] Non-admin users cannot delete their own access tokens

3 participants