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

Custom Unmarshaler for Consumer Identity #927

Merged
merged 4 commits into from
Sep 16, 2024

Conversation

fabianburth
Copy link
Contributor

What this PR does / why we need it

This PR adds a custom unmarshaler to the consumer identity type.

Adding this custom unmarshaler to the consumer identity allows a yaml specification containing a data type other string, e.g. a hostpath spec with a port. Previously, it would error if the user specified port: 5000 and instead, the user had to specify port: "5000".

Which issue(s) this PR fixes

Adding this custom unmarshaler to the consumer identity allows a yaml specification containing a data type other string, e.g. a hostpath spec with a port. Previously, it would error if the user specified `port: 5000` and instead, the user had to specify `port: "5000"`.
@fabianburth fabianburth added the area/ipcei Important Project of Common European Interest label Sep 16, 2024
frewilhelm
frewilhelm previously approved these changes Sep 16, 2024
ikhandamirov
ikhandamirov previously approved these changes Sep 16, 2024
api/credentials/internal/identity.go Outdated Show resolved Hide resolved
@fabianburth fabianburth dismissed stale reviews from ikhandamirov and frewilhelm via e774b4c September 16, 2024 13:31
@mandelsoft mandelsoft merged commit 2056314 into open-component-model:main Sep 16, 2024
14 checks passed
fabianburth added a commit that referenced this pull request Nov 7, 2024
<!-- markdownlint-disable MD041 -->
#### What this PR does / why we need it
Fix empty identity value regression.

The regression was introduced after this
[PR](#927) adding a
custom unmarshaler for consumer identities which allows to specify
values other than string (e.g. port: 5000), even though consumer
identity is a map[string]string.


#### Which issue(s) this PR fixes
<!--
Usage: `Fixes #<issue number>`, or `Fixes (paste link of issue)`.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ipcei Important Project of Common European Interest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants