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

[identity-providers][multi-env] Adapt role mapping screen for multi-environment #4803

Closed
phiz71 opened this issue Dec 15, 2020 · 1 comment · Fixed by gravitee-io/gravitee-management-rest-api#1363 or gravitee-io/gravitee-management-webui#1466
Closed

[identity-providers][multi-env] Adapt role mapping screen for multi-environment #4803

phiz71 opened this issue Dec 15, 2020 · 1 comment · Fixed by gravitee-io/gravitee-management-rest-api#1363 or gravitee-io/gravitee-management-webui#1466
Assignees
@gaetanmaisse @phiz71
Labels
cockpit & multi-env project: APIM type: enhancement
Projects
API Management
Milestone
APIM - 3.6.0

Comments

@phiz71
Copy link
Member

phiz71 commented Dec 15, 2020

Currently, role-mapping for environments will only be applied for 'DEFAULT' environment.
In a multi-env context, we should provide a way to configure different roles for different environments.
@phiz71 phiz71 added this to the APIM - 3.6.0 milestone Dec 15, 2020
@phiz71 phiz71 self-assigned this Feb 3, 2021
phiz71 added a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Feb 4, 2021
@phiz71
feat(management): allow to configure roleMapping on several environments
77ca8cb 
* handle backward compatibility, as the previous format in DB is ENVIRONMENT:ROLE_NAME and the new is ENVIRONMENT:ENV_ID:ROLE_NAME
* shared code between OAuth2AuthenticationResource and UserServiceImpl
* add a unit test in UserService

Closes gravitee-io/issues#4803
phiz71 added a commit to gravitee-io/gravitee-management-webui that referenced this issue Feb 4, 2021
@phiz71
feat(management): allow to configure roleMapping on several environments
2066743 
Closes gravitee-io/issues#4803
phiz71 added a commit to gravitee-io/gravitee-docs that referenced this issue Feb 4, 2021
@phiz71
docs: change gravitee.yml examples for roleMappings
e1d82b3 
Closes gravitee-io/issues#4803
This was referenced Feb 4, 2021
Merged
Merged
Merged
@gaetanmaisse gaetanmaisse self-assigned this Feb 5, 2021
@gaetanmaisse
Copy link
Contributor

I tested it locally but had some weird issues 🤔 to discuss

phiz71 added a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Feb 8, 2021
@phiz71
feat(management): allow to configure roleMapping on several environments
b52c20c 
* handle backward compatibility, as the previous format in DB is ENVIRONMENT:ROLE_NAME and the new is ENVIRONMENT:ENV_ID:ROLE_NAME
* shared code between OAuth2AuthenticationResource and UserServiceImpl
* add a unit test in UserService

Closes gravitee-io/issues#4803
phiz71 added a commit to gravitee-io/gravitee-management-webui that referenced this issue Feb 8, 2021
@phiz71
feat(management): allow to configure roleMapping on several environments
cd19e71 
Closes gravitee-io/issues#4803
gaetanmaisse pushed a commit to gravitee-io/gravitee-management-rest-api that referenced this issue Feb 8, 2021
@phiz71 @gaetanmaisse
feat(management): allow to configure roleMapping on several environments
a1967ce 
* handle backward compatibility, as the previous format in DB is ENVIRONMENT:ROLE_NAME and the new is ENVIRONMENT:ENV_ID:ROLE_NAME
* shared code between OAuth2AuthenticationResource and UserServiceImpl
* add a unit test in UserService

Closes gravitee-io/issues#4803
gaetanmaisse pushed a commit to gravitee-io/gravitee-management-webui that referenced this issue Feb 8, 2021
@phiz71 @gaetanmaisse
feat(management): allow to configure roleMapping on several environments
0af4d92 
Closes gravitee-io/issues#4803
plsstephenson pushed a commit to gravitee-io/gravitee-docs that referenced this issue Feb 16, 2021
@phiz71 @plsstephenson
docs: change gravitee.yml examples for roleMappings
e81ec9f 
Closes gravitee-io/issues#4803
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gaetanmaisse gaetanmaisse

@phiz71 phiz71

Labels
cockpit & multi-env project: APIM type: enhancement
Projects
API Management
Awaiting triage
Milestone
APIM - 3.6.0
2 participants
@gaetanmaisse @phiz71