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

fix: extra scope are not sending in passport #538

Closed
mmrraju opened this issue May 9, 2023 · 2 comments · Fixed by #539
Closed

fix: extra scope are not sending in passport #538

mmrraju opened this issue May 9, 2023 · 2 comments · Fixed by #539
Assignees
Labels

Comments

@mmrraju
Copy link

mmrraju commented May 9, 2023

Describe the issue

We are trying to send some extra scope in openid-client passport provider but scopes are not sending and granted except default ones
["openid","email","profile"].
According to ticket

Steps To Reproduce

Steps to reproduce the behavior:

  1. Install passport on my local server.
  2. Create a passport provider type: openid-client
  3. Creat an openid client on remote server.
  4. Provide client info in local passport providers options. Here scopes are ["openid","email","profile","phone","clientinfo"]
  5. For the local server We check "passport.log"
  6. For the RS we check "oxauth.log"

Expected behavior

For RS oxauth.log

  1. In here scope should be ["openid","email","profile","phone","clientinfo"]
  2. Here Granted scope should be ["openid","email","profile","phone","clientinfo"]

For LS passport.log

  1. In provider config scopes are ["openid","email","profile","phone","clientinfo"]
  2. Here response should be with scope ["openid","email","profile","phone","clientinfo"]

Actual behavior

For RS oxauth.log

  1. In "gluu.oxauth.authorize.ws.rs.AuthorizeRestWebServiceImpl" we see scope=["openid","email","profile"]
  2. Here Granted scope only ["openid","email","profile"]

For LS passport.log

  1. we get response with scope ["openid","email","profile"]

Desktop (please complete the following information):

  • OS: [e.g. Ubuntu20.04LTS]
  • Gluu version(4.4/4.5)
@mzico mzico added the bug label May 10, 2023
@crazedmeph
Copy link

@mzico @kdhttps any update on this one? This is holding me back from upgrading to version 4.5

@mzico
Copy link

mzico commented May 17, 2023

@crazedmeph : we are trying to do some final testing from this end before developer jump inside the code. @mmrraju

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

Successfully merging a pull request may close this issue.

4 participants