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.

Sign up for GitHub

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

Jump to bottom

fix: use authoriztion_url for authz request #47

Closed
duttarnab opened this issue Feb 8, 2022 · 3 comments
Closed

fix: use authoriztion_url for authz request #47

duttarnab opened this issue Feb 8, 2022 · 3 comments
Assignees
@duttarnab
Labels
bug Something isn't working comp-admin-ui Component affected by issue or PR

Comments

@duttarnab
Copy link
Contributor

  1. Currently following url is used for authorization.
    https://%(hostname)s/jans-auth/authorize.htm It should be changed to https://%(hostname)s/jans-auth/restv1/authorize.

  2. Remove acr_values from authoriztion_url .
@duttarnab duttarnab added bug Something isn't working comp-admin-ui Component affected by issue or PR labels Feb 8, 2022
@duttarnab duttarnab self-assigned this Feb 8, 2022
duttarnab added a commit that referenced this issue Feb 8, 2022
@duttarnab
fix: use authoriztion_url for authz request #47
8d794d5
duttarnab added a commit that referenced this issue Feb 8, 2022
@duttarnab
fix: use authoriztion_url for authz request #47
92f9ade
@duttarnab duttarnab mentioned this issue Feb 8, 2022
Merged
syntrydy added a commit that referenced this issue Feb 8, 2022
@syntrydy
Merge pull request #48 from GluuFederation/admin-ui-issue-47
952492e 
fix: use authoriztion_url for authz request #47
This was referenced Feb 8, 2022
Merged
Merged
@duttarnab
Copy link
Contributor Author

Done

@duttarnab
Copy link
Contributor Author

duttarnab commented Mar 3, 2022 •
edited
Loading

need to revert #2

  • Remove acr_values from authoriztion_url .

@duttarnab duttarnab reopened this Mar 3, 2022
duttarnab added a commit that referenced this issue Mar 3, 2022
@duttarnab
fix: use authoriztion_url for authz request #47
40b703c
@duttarnab duttarnab mentioned this issue Mar 3, 2022
Merged
mjatin-dev added a commit that referenced this issue Mar 3, 2022
@mjatin-dev
Merge pull request #120 from GluuFederation/admin-ui-issue-47
23675aa 
fix: use authoriztion_url for authz request #47
@mo-auto mo-auto mentioned this issue Mar 15, 2022
Merged
@duttarnab
Copy link
Contributor Author

Done

This was referenced Jul 14, 2022
Merged
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@duttarnab duttarnab

Labels
bug Something isn't working comp-admin-ui Component affected by issue or PR
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@duttarnab