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

Enable portal helm chart to work with SSI DIM Wallet Stub #499

Open
5 tasks
evegufy opened this issue Jan 30, 2025 · 1 comment
Open
5 tasks

Enable portal helm chart to work with SSI DIM Wallet Stub #499

evegufy opened this issue Jan 30, 2025 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@evegufy
Copy link
Contributor

evegufy commented Jan 30, 2025

Description

This issue is the result of the discussion here eclipse-tractusx/tractus-x-umbrella#197 (comment)

Background: according the currently implemented concept, the did document is only to be located in portal database (and accessible via portal-backend api), that's the reason why the portal doesn't work out of the box with ssi-dim-wallet-stub, which isn't hosted unter portal-backend domain.

One could think of doing a change to the portal helm chart, essentially providing an option to maintain the entire did document address (domain and path) in the values file, with the exception to the /did.json at the end, which comes from the portal-backend code. IMPORTANT: the default value for the did document baselocation should still be portal-backend address + did document path see in order to follow the implemented concept.

Acceptance Criteria

  • did document baselocation still defaults to portal-backend address + did document path
  • in the values.yaml, a parameter dim.didDocumentAddress was introduced which is enabled in the jobs and deployments which currently contain the did document baselocation
  • only if the dim.didDocumentAddress is filled, it is used else defaulting to current logic (portal-backend address + did document path)
  • install comments were added to explain this logic (only fill if you want to differ from the default portal-backend address + did document path)
  • one could also move the didDocumentPath into the jobs and deployments because enabling the option to change that value makes only limited sense considering implementation

Additional Information

detected in v2.3.0

@evegufy
Copy link
Contributor Author

evegufy commented Feb 10, 2025

please checkout from the rc branch https://github.com/eclipse-tractusx/portal/tree/release-candidate and raise PR back to it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: BACKLOG
Development

No branches or pull requests

2 participants