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

[4.2.x] fix: handle json mapping exception #7014

Merged
merged 1 commit into from Mar 25, 2024
Merged

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Mar 25, 2024

This is an automatic copy of pull request #7011 done by Mergify.


This is an automatic copy of pull request #7006 done by Mergify.


Issue

https://gravitee.atlassian.net/browse/APIM-3795

Description

handle json mapping exception to throw only detail message without stack trace

Additional context


Cherry-pick of 48ad60c has failed:

On branch mergify/bp/4.0.x/pr-7006
Your branch is up to date with 'origin/4.0.x'.

You are currently cherry-picking commit 48ad60c635.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   gravitee-apim-e2e/api-test/src/applications/applications.spec.ts
	modified:   gravitee-apim-e2e/api-test/src/portal/portal-applications.spec.ts
	modified:   gravitee-apim-e2e/lib/fixtures/management/ApplicationsFaker.ts
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-management/gravitee-apim-rest-api-management-rest/src/main/java/io/gravitee/rest/api/management/rest/provider/JsonMappingExceptionMapper.java
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-portal/gravitee-apim-rest-api-portal-rest/src/main/java/io/gravitee/rest/api/portal/rest/provider/JsonMappingExceptionMapper.java
	modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-portal/gravitee-apim-rest-api-portal-rest/src/main/java/io/gravitee/rest/api/portal/rest/resource/GraviteePortalApplication.java

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-management/gravitee-apim-rest-api-management-rest/src/main/java/io/gravitee/rest/api/management/rest/resource/GraviteeManagementApplication.java

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally


📚  View the storybook of this branch here


Cherry-pick of 6f13d3b has failed:

On branch mergify/bp/4.2.x/pr-7011
Your branch is up to date with 'origin/4.2.x'.

You are currently cherry-picking commit 6f13d3b725.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   gravitee-apim-e2e/api-test/src/apis/imports/mapi-v2/import-api-definition/api-v4-import-only-api.spec.ts
	modified:   gravitee-apim-e2e/api-test/src/applications/mapi-v1/applications.spec.ts
	modified:   gravitee-apim-e2e/api-test/src/portal/mapi-v1/portal-applications.spec.ts
	modified:   gravitee-apim-e2e/lib/fixtures/management/ApplicationsFaker.ts
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-management-v2/gravitee-apim-rest-api-management-v2-rest/src/main/java/io/gravitee/rest/api/management/v2/rest/exceptionMapper/JsonMappingExceptionMapper.java
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-management/gravitee-apim-rest-api-management-rest/src/main/java/io/gravitee/rest/api/management/rest/provider/JsonMappingExceptionMapper.java
	modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-management/gravitee-apim-rest-api-management-rest/src/main/java/io/gravitee/rest/api/management/rest/resource/GraviteeManagementApplication.java
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-portal/gravitee-apim-rest-api-portal-rest/src/main/java/io/gravitee/rest/api/portal/rest/provider/JsonMappingExceptionMapper.java
	modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-portal/gravitee-apim-rest-api-portal-rest/src/main/java/io/gravitee/rest/api/portal/rest/resource/GraviteePortalApplication.java

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-management-v2/gravitee-apim-rest-api-management-v2-rest/src/main/java/io/gravitee/rest/api/management/v2/rest/GraviteeManagementV2Application.java

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally


📚  View the storybook of this branch here

@mergify mergify bot added the conflicts label Mar 25, 2024
@mergify mergify bot requested a review from a team as a code owner March 25, 2024 10:18
Copy link
Author

mergify bot commented Mar 25, 2024

Cherry-pick of 6f13d3b has failed:

On branch mergify/bp/4.2.x/pr-7011
Your branch is up to date with 'origin/4.2.x'.

You are currently cherry-picking commit 6f13d3b725.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   gravitee-apim-e2e/api-test/src/apis/imports/mapi-v2/import-api-definition/api-v4-import-only-api.spec.ts
	modified:   gravitee-apim-e2e/api-test/src/applications/mapi-v1/applications.spec.ts
	modified:   gravitee-apim-e2e/api-test/src/portal/mapi-v1/portal-applications.spec.ts
	modified:   gravitee-apim-e2e/lib/fixtures/management/ApplicationsFaker.ts
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-management-v2/gravitee-apim-rest-api-management-v2-rest/src/main/java/io/gravitee/rest/api/management/v2/rest/exceptionMapper/JsonMappingExceptionMapper.java
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-management/gravitee-apim-rest-api-management-rest/src/main/java/io/gravitee/rest/api/management/rest/provider/JsonMappingExceptionMapper.java
	modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-management/gravitee-apim-rest-api-management-rest/src/main/java/io/gravitee/rest/api/management/rest/resource/GraviteeManagementApplication.java
	new file:   gravitee-apim-rest-api/gravitee-apim-rest-api-portal/gravitee-apim-rest-api-portal-rest/src/main/java/io/gravitee/rest/api/portal/rest/provider/JsonMappingExceptionMapper.java
	modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-portal/gravitee-apim-rest-api-portal-rest/src/main/java/io/gravitee/rest/api/portal/rest/resource/GraviteePortalApplication.java

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   gravitee-apim-rest-api/gravitee-apim-rest-api-management-v2/gravitee-apim-rest-api-management-v2-rest/src/main/java/io/gravitee/rest/api/management/v2/rest/GraviteeManagementV2Application.java

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

@Okhelifi Okhelifi changed the title [4.2.x] [4.0.x] fix: handle json mapping exception [4.2.x] fix: handle json mapping exception Mar 25, 2024
@Okhelifi Okhelifi force-pushed the mergify/bp/4.2.x/pr-7011 branch 2 times, most recently from 4df440b to bd9c3cc Compare March 25, 2024 11:06
…ut stack trace

(cherry picked from commit 48ad60c)

(cherry picked from commit 6f13d3b)
@Okhelifi Okhelifi merged commit 3cf6aba into 4.2.x Mar 25, 2024
37 of 38 checks passed
@Okhelifi Okhelifi deleted the mergify/bp/4.2.x/pr-7011 branch March 25, 2024 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant