Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mbayou
Copy link
Contributor

@mbayou mbayou commented Sep 6, 2025

Yesterday’s PR was flawed, my mistake. While testing the integration this morning, I realised I had added the apiKey in the body instead of as a request parameter. This update should fix the issue.

PS: I’ve been trying to package it locally to test directly with my code, but after an hour I keep running into issues with the licence plugin failing to execute. Any chance you could run it quickly on your side, or share the instructions to get it working? Sorry about that, it feels a bit amateurish, but I need to move on to other tasks today. I’m fairly confident the mistake from last time is fixed after re-reading the documentation, but I would have preferred to test it properly.

Mathieu Bayou and others added 5 commits September 5, 2025 15:40
We do not need to null check as Eulerstream verifies anyway so we can just pass whatever the value is.
Revert to your method, but using Header instead of Param, since HttpClient does not allow null values!
@kohlerpop1
Copy link
Collaborator

kohlerpop1 commented Sep 6, 2025

Yesterday’s PR was flawed, my mistake. While testing the integration this morning, I realised I had added the apiKey in the body instead of as a request parameter. This update should fix the issue.

PS: I’ve been trying to package it locally to test directly with my code, but after an hour I keep running into issues with the licence plugin failing to execute. Any chance you could run it quickly on your side, or share the instructions to get it working? Sorry about that, it feels a bit amateurish, but I need to move on to other tasks today. I’m fairly confident the mistake from last time is fixed after re-reading the documentation, but I would have preferred to test it properly.

You can just comment out the license plugin in the pom.xml and it will run. But none the less, I tested this and was able to get passed verification.

@kohlerpop1 kohlerpop1 merged commit d325dff into jwdeveloper:master Sep 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants