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

[KEYCLOAK-4932] - Spring Boot quickstart using fine-grained permissions #26

Merged
merged 2 commits into from Jun 6, 2017

Conversation

Projects
None yet
3 participants
@pedroigor
Copy link
Contributor

commented May 18, 2017

@pedroigor pedroigor force-pushed the pedroigor:KEYCLOAK-4932 branch 2 times, most recently from e280a68 to 35f2112 May 19, 2017

keycloak.credentials.secret=secret
product.service.url=http://localhost:8081/products
keycloak.policy-enforcer-config.on-deny-redirect-to=/accessDenied
keycloak.securityConstraints[1].authRoles[0] = user

This comment has been minimized.

Copy link
@sebastienblanc

sebastienblanc May 24, 2017

Contributor

Not a big deal but it should be securityConstraints[0] instead of [1]

@sebastienblanc

This comment has been minimized.

Copy link
Contributor

commented May 24, 2017

@pedroigor Could you rebase this PR ? We just merged some changes that moves some classes around ;)

abstractj pushed a commit to abstractj/keycloak-quickstarts that referenced this pull request May 31, 2017

Merge pull request keycloak#26 from abstractj/RHSSO-811
[RHSSO-811] Error processing request after login to app-authz-vanilla

@pedroigor pedroigor force-pushed the pedroigor:KEYCLOAK-4932 branch from 35f2112 to 3667467 Jun 1, 2017

@pedroigor

This comment has been minimized.

Copy link
Contributor Author

commented Jun 1, 2017

@sebastienblanc Sure. But after the merge the quickstart is not working anymore. Even tests are broken. Will see what I need to change.

@pedroigor

This comment has been minimized.

Copy link
Contributor Author

commented Jun 2, 2017

@sebastienblanc @abstractj PR rebased. I've also added some more docs and removed unnecessary code.

Tks.

@abstractj

This comment has been minimized.

Copy link
Contributor

commented Jun 5, 2017

@sebastienblanc

This comment has been minimized.

Copy link
Contributor

commented Jun 6, 2017

👀

@sebastienblanc sebastienblanc merged commit d58b3a0 into keycloak:master Jun 6, 2017

1 check failed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.