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

Incompatibility with Hashicorp Vault 1.18.1, last working version is 1.16.1 #1772

Closed
lgblaumeiser opened this issue Jan 28, 2025 · 2 comments · Fixed by #1796
Closed

Incompatibility with Hashicorp Vault 1.18.1, last working version is 1.16.1 #1772

lgblaumeiser opened this issue Jan 28, 2025 · 2 comments · Fixed by #1796
Assignees
Labels
bug Something isn't working

Comments

@lgblaumeiser
Copy link
Contributor

Describe the bug

As I recently found out, there is a general incompatibility with the Hashicorp Vault 1.18.1. The bug is actually on there side, but current connector versions are not able to work with this vault version. Last known version that works is 1.16.1.

Issue is, that the way, the upstream connector creates the url for a rest call, it uses encoded slashes, resp. %2F. According to the corresponding web standard a system should properly work with plain and encoded slashes, but version 1.18.1 does not. This way actually encountered in the test of the BDRS service, as the tests took the latest version of the vault which failed at some point consistently.

To Reproduce

  1. Configure the deployment tests, of e.g., the BDRS, to use a vault 1.18.1 (resp. the corresponding helm chart)
  2. Try to run the tests
  3. Observe stange IOExceptions which indicate that an html page was returned at some point instead of a JSON object

Expected behavior

As the error is on the vault side, the only means is to use a working vault implementation

@lgblaumeiser lgblaumeiser added bug Something isn't working triage all new issues awaiting classification labels Jan 28, 2025
@ndr-brt
Copy link
Contributor

ndr-brt commented Feb 3, 2025

JFYI, upstream the Hashicorp version used for integration tests has been bumped to 1.18.3, so this one is compatible with EDC, I don't know about 1.18.1, but should this considered a bug then?

@lgblaumeiser lgblaumeiser self-assigned this Feb 4, 2025
@lgblaumeiser lgblaumeiser removed the triage all new issues awaiting classification label Feb 4, 2025
@lgblaumeiser
Copy link
Contributor Author

lgblaumeiser commented Feb 4, 2025

Solution: Make a remark in the Readme that this version will not work with the EDC as discussed in the weekly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants