Skip to content

Conversation

@SamBobBarnes
Copy link

@SamBobBarnes SamBobBarnes commented Oct 18, 2025

add authParams param
correct existing documentation

What is this PR and why do we need it?

Currently the login process fails and reports error=invalid_request&error_reason=missing_response_type&error_description=The+request+is+missing+a+required+parameter%3A+response_type when using the SDK on angular and vue #159.

Adding this param allows developers to add the repsonse_type query parameter that is required according to the docs.

Pre-Merge Checklist (if applicable)

  • Unit and Feature tests have been added/updated for logic changes, or there is a justifiable reason for not doing so.
    No logic was changed. This parameter was added to a type def and is doing the same as is correspondent in the react sdk.

correct existing documentation
@kmaida
Copy link

kmaida commented Oct 31, 2025

Can you please provide repro? Adding an arbitrary authParams parameter is likely avoidable. response_type is an existing parameter on the oauth2/authorize endpoint. However, you wouldn't pass it inside another parameter, you'd include it as a parameter directly on the oauth2/authorize request.

What does your authorization request URI look like?

It should resemble this: http://localhost:9011/oauth2/authorize?client_id={client_id}&redirect_uri={redirect_uri}&response_type=code

@kmaida kmaida self-assigned this Oct 31, 2025
@kmaida kmaida added the question Further information is requested label Oct 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants