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

Fix user token API endpoints #27040

Closed
wants to merge 1 commit into from
Closed

Conversation

JakobDev
Copy link
Contributor

The user token endpoints use currently the /users/{username}/tokens route. So they take a Username as Argument in the Path. But that Argument is never used. Those endpoints will always use the authenticated user, no matter what name you put in. This PR changes the route to /user/tokens. The old route will still work, but is marked as deprecated.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 12, 2023
@pull-request-size pull-request-size bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Sep 12, 2023
@github-actions github-actions bot added type/docs This PR mainly updates/creates documentation modifies/api This PR adds API routes or modifies them labels Sep 12, 2023
silverwind added a commit that referenced this pull request Sep 18, 2023
Fix #26234
close #26323
close #27040

---------

Co-authored-by: silverwind <me@silverwind.io>
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Dec 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/api This PR adds API routes or modifies them size/L Denotes a PR that changes 100-499 lines, ignoring generated files. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants