feat: add support for state param for OAuth2 Authorization Code flow
Issue Reference
https://github.com/usebruno/bruno/issues/2275
Description
Adding support for state param which is expected on OAuth2 authorization code flow
- There is also another PR trying to solve same issue but generating state param on the fly, but that is still not merged, so created this in case this is expected, any of two can be merged and other closed.
- This PR should save the state and reload as well and can allow user to pass whatever state they wants to
Contribution Checklist:
- [x] The pull request only addresses one issue or adds one feature.
- [x] The pull request does not introduce any breaking changes
- [x] I have added screenshots or gifs to help explain the change if applicable.
- [x] I have read the contribution guidelines.
- [x] Create an issue and link to the pull request.
Note: Keeping the PR small and focused helps make it easier to review and merge. If you have multiple changes you want to make, please consider submitting them as separate pull requests.
Publishing to New Package Managers
Please see here for more information.
Github issue: https://github.com/usebruno/bruno/issues/2275
@lohxt1 Assigning this to you
Lets make it a priority to have this merged early next week.
Fixed reviews! Feel free to take a look
Let me know if any more changes are needed to this PR @lohxt1
Merged!
Thanks @dhananjaykadam for working on this! Thanks @pietrygamat @lohxt1 for the review.
This will go out next Tuesday, June 4th 2024.