Hello contributors :slightly_smiling_face: I face...
# contributors
c
Hello contributors 🙂 I faced an issue while using Hydra ApiClient with generated project hydra-client-java. It is related to a NPE while building PUT request with empty body after calling acceptLogoutRequest. Since ApiClient is generated by openapi-generator the bug comes from their base code. They have fixed it in the 6.0.0 release. So I would like to bump the openapi-generator version from 5.4.0 to 6.0.0 to include the fix to java clients. I have created a bug ticket here : https://github.com/ory/sdk/issues/191 I also have create a fork and a draft pull request to try to fix the issue : https://github.com/ory/sdk/pull/192 I found where to increase the version for java clients. But I am having trouble trying to test it locally and be sure that the major release 6.0.0 does not break stuff. Locally I forked the project and only managed to make the generation work by using :
Copy code
docker run --platform linux/amd64 --mount type=bind,source="$(pwd)",target=/project -it oryd/sdk:latest /bin/sh

export FORCE_VERSION=v1.11.0
export FORCE_PROJECT=hydra # or hydra or something else
cd /project

./scripts/generate.sh
But when I call the ./scripts/test.sh script I have this error : ./scripts/test.sh: line 88: mix: command not found On the other side I have issues when I run mvn test-compile directly under /clients/hydra/java. I would like to see with you if there are best practices for testing version upgrades ^^
h
Hi Lucas, thank you! I think the info that they fixed an important bug is enough for me to get it merged 🙂 For breaking changes, you can also always look at the release change log from open api generator itself: https://github.com/OpenAPITools/openapi-generator/releases/tag/v6.0.0#java
👍 1
I answered your question on the GH PR in full length
c
Thanks I will have a look 🙂