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

/user/userGetCurrent should be always accessible #33583

Open
AdamMajer opened this issue Feb 13, 2025 · 2 comments
Open

/user/userGetCurrent should be always accessible #33583

AdamMajer opened this issue Feb 13, 2025 · 2 comments
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.

Comments

@AdamMajer
Copy link
Contributor

Feature Description

The endpoint /user/userGetCurrent currently requires [read:user] scope permissions. Would it make sense to make this universally accessible for all tokens? For example, I do not want access to information about other users or tokens, but just information about current token account.

Alternatively, maybe there should be a distinction about getting read permissions to current user vs. read permissions to all users?

Screenshots

No response

@AdamMajer AdamMajer added the type/proposal The new feature has not been accepted yet but needs to be discussed first. label Feb 13, 2025
@lunny
Copy link
Member

lunny commented Feb 13, 2025

But when you want to give others a token but it can only read repositories or issues, wouldn't it be more strict?

@AdamMajer
Copy link
Contributor Author

The main concern here is when you have a token, and can read repositories or issues, you don't know which comments are made by the owner of the token. At least not by just having a token. This is somewhat important if you use issue comments as a data store.

So, I agree with you that it's more strict, but we don't have separate permission to just read current user data. There's only ability to enable reading all users data. Adding more fine grain control to only read current user data would be more than enough.

This is mostly a nit issue and not very important.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/proposal The new feature has not been accepted yet but needs to be discussed first.
Projects
None yet
Development

No branches or pull requests

2 participants