usebruno / bruno

Opensource IDE For Exploring and Testing Api's (lightweight alternative to postman/insomnia)
https://www.usebruno.com/
MIT License
21.54k stars 1.01k forks source link

fix: Pass the scope conditionally for OAuth2 Password Credentials #2447

Closed nicolasnobelis closed 1 week ago

nicolasnobelis commented 2 weeks ago

The scope param is always passed (when according to the spec, it is optional).

The server responds with

{ "error": "invalid_scope", "error_description": "Invalid scopes: " }

Fixes: https://github.com/usebruno/bruno/issues/2446.

pietrygamat commented 1 week ago

The same is true for other oauth2 grant types. Perhaps it's worth updating different ones as well?

helloanoop commented 1 week ago

Merged!

Thanks for working on this @nicolasnobelis ! Thanks for the review @pietrygamat @lohxt1 !