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

Unclear documentation / returned errors regarding required authentication scopes #131

Open
xeniorn opened this issue Nov 23, 2023 · 0 comments

Comments

@xeniorn
Copy link

xeniorn commented Nov 23, 2023

Required scope for the YouTrack BearerTokenConnection bearer token is unclear.

Expected behavior

Report which of the two auth scopes are missing

Actual behavior

Generic error response when there is insufficient auth.

I created a bearer token connection with "YouTrack" and "YouTrack Administration" tokens separately, got rather unhelpful errors on attempts to create any service to "check the response". After some digging through library code, I found that under "GetAuthenticatedApiClient" auth is carried out twice, once via "_youTrackClient.UsersMeAsync("id,guest");" and once with "_youTrackClient.HubApiUserGetAsync("me", "guest");". Unless both work, the same message is given.

In the current implementation, it's also not possible to have a YouTrack-only scope, must include also the admin part. Possibly it would be better if they would individually work as well, just giving a error if you try to use the functions outside the scope covered by the provided bearer.

If the current implementation is kept, would be good if it was written in an obvious way in the docu that both scopes are required for the library to work.

Cheers

J

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant