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

fix: scim user management endpoint failing due to conflict with user … #1181

Merged
merged 1 commit into from
Apr 8, 2022

Conversation

pujavs
Copy link
Contributor

@pujavs pujavs commented Apr 7, 2022

Fix for two issues in jenkins build for jans-config-api

1) jans-config-api-server :

Error: CAUSED BY:org.jboss.resteasy.spi.UnhandledException: com.fasterxml.jackson.databind.JsonMappingException: Index 0 out of bounds for length 0 (through reference chain: java.util.ArrayList[0]->io.jans.as.common.model.common.User[""customAttributes""]->java.util.ArrayList[4]->io.jans.orm.model.base.CustomObjectAttribute[""displayValue""
Issue: io.jans.orm.model.base.CustomObjectAttribute.java:getDisplayValue() - accessing list element without checking the size
Fix: io.jans.orm.model.base.CustomObjectAttribute.java:getDisplayValue() - added checked access for list element

2) scim-plugin

Error: src.test.resources.feature.scim.user.scim-user: scim-user.feature:33 - javascript evaluation failed: mainUrl + '/' +response.Resources[0].id, TypeError: Cannot read property ""0"" from undefined in at line number 1
scim-user.feature:68 - status code was: 400, expected: 200, response time: 121, url: https://jenkins-build.jans.io/jans-config-api/scim/user/null, response: com.fasterxml.jackson.databind.exc.MismatchedInputException: Cannot construct instance of java.util.LinkedHashMap (although at least one Creator exists): no String-argument constructor/factory method to deserialize from String value ('2022-04-07T09:54:13')
Issue: Conflict of endpoint path with newly added user managment service path
Fix: Renamed scim service path

@mo-auto mo-auto added comp-jans-cli-tui Component affected by issue or PR comp-jans-config-api Component affected by issue or PR comp-jans-orm Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Apr 7, 2022
@sonarcloud
Copy link

sonarcloud bot commented Apr 7, 2022

[jans-cli] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Apr 7, 2022

[orm] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

@sonarcloud
Copy link

sonarcloud bot commented Apr 7, 2022

[jans-config-api-parent] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@yuriyz yuriyz merged commit 8ee47a0 into main Apr 8, 2022
@yuriyz yuriyz deleted the jans-build-issue branch April 8, 2022 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp-jans-cli-tui Component affected by issue or PR comp-jans-config-api Component affected by issue or PR comp-jans-orm Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants