Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: handle authorization_details in back_channel_login #695

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

Merged
merged 14 commits into from
Jun 4, 2025

Conversation

kishore7snehil
Copy link
Contributor

@kishore7snehil kishore7snehil commented Jun 2, 2025

Changes

  • Add support for authorization_details parameter in back_channel_login method
  • Serialize authorization_details to JSON strings before sending requests
  • Update docstrings to clarify expected formats for authorization_details
  • Modify tests to expect serialized JSON strings for authorization_details

References

Testing

  • This change adds test coverage

  • This change has been tested on the latest version of the platform/language or why not

Contributor Checklist

@kishore7snehil kishore7snehil requested a review from a team as a code owner June 2, 2025 15:00
@kishore7snehil kishore7snehil changed the title fix: support authorization_details in back_channel_login fix: handle authorization_details in back_channel_login Jun 2, 2025
@kishore7snehil kishore7snehil merged commit ba032e8 into master Jun 4, 2025
9 checks passed
@kishore7snehil kishore7snehil deleted the fix/ciba_request branch June 4, 2025 08:18
@kishore7snehil kishore7snehil mentioned this pull request Jun 5, 2025
kishore7snehil added a commit that referenced this pull request Jun 5, 2025
**Added**
- chore: merge community PRs – bugfixes, features, and dependency
upgrades [\#696](#696)
([kishore7snehil](https://github.com/kishore7snehil))

**Fixed**
- fix: handle `authorization_details` in back_channel_login
[\#695](#695)
([kishore7snehil](https://github.com/kishore7snehil))
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

Successfully merging this pull request may close these issues.

3 participants