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

#3033 Fixed HttpMessageNotWritableException: #3041

Merged
merged 3 commits into from
Nov 18, 2024

Conversation

Limraj
Copy link
Collaborator

@Limraj Limraj commented Oct 29, 2024

  • Added jackson-datatype-jdk8-2.17.2.jar for default deserializers/serializers types from java 8;
  • XssUtils.validate: If field null or empty then true;
  • Configure objectMapper: enabled CLOSE_CLOSEABLE, disabled: DEFAULT_VIEW_INCLUSION, FAIL_ON_UNKNOWN_PROPERTIES, WRITE_DATES_AS_TIMESTAMPS, FAIL_ON_NULL_FOR_PRIMITIVES;
  • Fixed code 400 in REST API /api/datapoint/ PUT/POST/GET, problem with JsonTypeInfo.As.EXTERNAL_PROPERTY;

- Added jackson-datatype-jdk8-2.17.2.jar for default deserializers/serializers types from java 8;
- XssUtils.validate: If field null or empty then true;
- Configure objectMapper: enabled CLOSE_CLOSEABLE, disabled: DEFAULT_VIEW_INCLUSION, FAIL_ON_UNKNOWN_PROPERTIES, WRITE_DATES_AS_TIMESTAMPS, FAIL_ON_NULL_FOR_PRIMITIVES;
- Fixed code 400 in REST API /api/datapoint/ PUT/POST/GET, problem with JsonTypeInfo.As.EXTERNAL_PROPERTY;
@Limraj Limraj added this to the 2.8.0 milestone Oct 29, 2024
@Limraj Limraj requested a review from SoftQ as a code owner October 29, 2024 09:56
Copy link

github-actions bot commented Oct 29, 2024

Java Script Mocha Unit Test Results

268 tests   268 ✅  3s ⏱️
 70 suites    0 💤
  1 files      0 ❌

Results for commit cbf6a97.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Oct 29, 2024

Java JUnit Test Results

2 517 tests   2 517 ✅  43s ⏱️
  116 suites      0 💤
  116 files        0 ❌

Results for commit cbf6a97.

♻️ This comment has been updated with latest results.

- revert: DataPointJson.java, DataPointLocatorJson.java
@Limraj Limraj requested a review from Patrykb0802 November 6, 2024 19:54
Copy link
Contributor

@Patrykb0802 Patrykb0802 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked json data point importing, no database query errors occuring now.

@Limraj Limraj merged commit 6d585aa into release/2.8.0 Nov 18, 2024
8 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.

2 participants