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.

Sign up for GitHub

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

Jump to bottom

Rest API: accept unknown properties for forward compatibility #8

Closed
yvespp opened this issue Dec 3, 2016 · 1 comment
Closed

Rest API: accept unknown properties for forward compatibility #8

yvespp opened this issue Dec 3, 2016 · 1 comment
Milestone
Version 1.17.3

Comments

@yvespp
Copy link
Member

yvespp commented Dec 3, 2016 •
edited
Loading

The Rest API should accept properties it doesn't know without throwing errors. This enables clients to send data that is not yet used by AMW.

In JBoss this can be done via Jackson Config
Per class:

@org.codehaus.jackson.annotate.JsonIgnoreProperties(ignoreUnknown = true)
public class FooBar {

or globaly:

objectMapper.configure(DeserializationConfig.Feature.FAIL_ON_UNKNOWN_PROPERTIES, false);

This however the Jackson config binds AMW to JBoss (more)...
@yvespp yvespp changed the title Rest API: accept unknown keys for forward compatibility Rest API: accept unknown properties for forward compatibility Dec 3, 2016
@RetGal
Copy link
Contributor

RetGal commented Dec 13, 2016

The annotations are ignored, EAP 6.4 uses jackson-1.9.9 but DeserializationConfig.Feature needs 1.9.13 or newer. We have to wait for EAP 7..

@RetGal RetGal mentioned this issue Jul 4, 2018
Merged
@RetGal RetGal added this to the Version 1.17.3 milestone Jul 4, 2018
mburri added a commit that referenced this issue Oct 6, 2020
@mburri
migration to bootstrap 4 - loading indicator and notification (step #8)
addfbd5
mburri added a commit that referenced this issue Oct 15, 2020
@mburri
migration to bootstrap 4 - loading indicator and notification (step #8)
52e708f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
Version 1.17.3
Development

No branches or pull requests
2 participants
@yvespp @RetGal