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

feat(rest.identity.provider): Added REST Identity API V2 #5201

Merged

Conversation

salvatore-coppola
Copy link
Contributor

This PR add a new version of REST Identity levereging the new IdentityService and PasswordStrenghtVerificationService

@MMaiero
Copy link
Contributor

MMaiero commented Apr 12, 2024

Something is broken in the PR as it fails with the following error:

[INFO] --- tycho-compiler-plugin:1.7.0:compile (default-compile) @ org.eclipse.kura.request.handler.jaxrs ---
[WARNING] Parameter 'useProjectSettings' is set to true, but preferences file '/home/jenkins/agent/workspace/multibranch_PR-5201/kura/kura/org.eclipse.kura.request.handler.jaxrs/.settings/org.eclipse.jdt.core.prefs' could not be found!
[INFO] Compiling 8 source files to /home/jenkins/agent/workspace/multibranch_PR-5201/kura/kura/org.eclipse.kura.request.handler.jaxrs/target/classes
[WARNING] /home/jenkins/agent/workspace/multibranch_PR-5201/kura/kura/org.eclipse.kura.request.handler.jaxrs/src/main/java/org/eclipse/kura/request/handler/jaxrs/DefaultExceptionHandler.java:[63] 
	final Response response = e.getResponse();
	      ^^^^^^^^
Response is a raw type. References to generic type Response<T> should be parameterized

Please check

@MMaiero
Copy link
Contributor

MMaiero commented Apr 12, 2024

better, but still issues building

@salvatore-coppola
Copy link
Contributor Author

Merged in any case because failed tests are not related to PR.

@salvatore-coppola salvatore-coppola merged commit ab254e3 into eclipse-kura:develop Apr 17, 2024
2 of 3 checks passed
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.

4 participants