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

IRS OAuth2 Config - separate credential config for each RestTemplate #378

Closed
1 task
ds-jhartmann opened this issue Jan 22, 2024 · 1 comment
Closed
1 task

IRS OAuth2 Config - separate credential config for each RestTemplate #378

ds-jhartmann opened this issue Jan 22, 2024 · 1 comment
Assignees
@ds-ext-kmassalski
Labels
hardening measurements to increase resiliency

Comments

@ds-jhartmann
Copy link
Contributor

ds-jhartmann commented Jan 22, 2024 •
edited by mkanal
Loading

As IRS-Administrator,
I want be able to configure a technical user for each service,
so that I am not relying on special technical users.

Hints / Details

With the new Portal, we need to create a Technical User for each System, we interact with. That means that we need a separate User for

  • Semantic Model Management (user configured via spring.security.oauth2.client.registration.common)
  • BPDM Pool (user configured via spring.security.oauth2.client.registration.common)
  • Dataspace Discovery (user configured via spring.security.oauth2.client.registration.portal)
  • Identity Wallet Management (only configured in EDC, not relevant for IRS)

With the current configuration of IRS it is not possible to set a separate User for BPDM Pool, so a change in implementation is required.

Outcome / Acceptance Criteria

Outcome

  • ...

Acceptance Criteria

  • configuration for different backend services credentials is possible

Out of Scope

  • ...
@github-project-automation github-project-automation bot moved this to inbox in IRS Jan 22, 2024
@ds-jhartmann ds-jhartmann mentioned this issue Jan 22, 2024
Closed
14 tasks
@jzbmw jzbmw added the hardening measurements to increase resiliency label Jan 23, 2024
@mkanal mkanal moved this from inbox to backlog in IRS Jan 23, 2024
@jzbmw jzbmw moved this from backlog to next in IRS Jan 23, 2024
@ds-jhartmann
Copy link
Contributor Author

Planning 2:

  • rename existing client registrations to match the service they are used for

@ds-ext-kmassalski ds-ext-kmassalski self-assigned this Jan 23, 2024
@ds-ext-kmassalski ds-ext-kmassalski moved this from next to wip in IRS Jan 23, 2024
@ds-ext-kmassalski ds-ext-kmassalski moved this from wip to review in IRS Jan 25, 2024
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
feat(impl):[#378] separate credentials config
b5d0fb1
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
feat(impl):[#378] separate credentials config
eaf6f46
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
feat(impl):[#378] changelog
f933ca8
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
feat(impl):[#378] changelog
3223e42
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
Merge pull request #737 from catenax-ng/feature/378-separate-credenti…
6998ee8 
…als-config

feat(impl):[#378] separate credentials config
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
Revert "feat(impl):[#378] separate credentials config"
b2c3090
ds-jhartmann pushed a commit that referenced this issue Jan 29, 2024
@ds-ext-kmassalski
Merge pull request #741 from catenax-ng/revert-737-feature/378-separa…
bd9168f 
…te-credentials-config

Revert "feat(impl):[#378] separate credentials config"
@jzbmw jzbmw moved this from review to done in IRS Feb 5, 2024
@jzbmw jzbmw closed this as completed Feb 5, 2024
ds-jhartmann pushed a commit that referenced this issue Feb 7, 2024
@ds-ext-kmassalski
Revert "Revert "feat(impl):[#378] separate credentials config""
ff326e0
ds-jhartmann added a commit that referenced this issue Feb 7, 2024
@ds-jhartmann
Merge pull request #742 from catenax-ng/revert-741-revert-737-feature…
2d7ad3f 
…/378-separate-credentials-config

Revert "Revert "feat(impl):[#378] separate credentials config""
ds-jhartmann added a commit that referenced this issue Feb 7, 2024
@ds-jhartmann
Revert "Revert "Revert "feat(impl):[#378] separate credentials config"""
4c28b29
ds-jhartmann added a commit that referenced this issue Feb 7, 2024
@ds-jhartmann
Merge pull request #758 from catenax-ng/revert-742-revert-741-revert-…
d7bfe06 
…737-feature/378-separate-credentials-config

Revert "Revert "Revert "feat(impl):[#378] separate credentials config"""
ds-jhartmann added a commit that referenced this issue Feb 7, 2024
@ds-jhartmann
Revert "Revert "Revert "Revert "feat(impl):[#378] separate credential…
91b91e1 
…s config""""
ds-jhartmann added a commit that referenced this issue Feb 7, 2024
@ds-jhartmann
feat(i-api):[#378] Fix test config
c16476a
ds-jhartmann added a commit that referenced this issue Feb 7, 2024
@ds-jhartmann
Merge pull request #759 from catenax-ng/revert-758-revert-742-revert-…
40ae1c3 
…741-revert-737-feature/378-separate-credentials-config

feat(impl):[#378] separate credentials config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ds-ext-kmassalski ds-ext-kmassalski

Labels
hardening measurements to increase resiliency
Projects
IRS
Status: done
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jzbmw @ds-jhartmann @ds-ext-kmassalski