data-prepper icon indicating copy to clipboard operation
data-prepper copied to clipboard

Fix enum deserialization for geoip and dissect processors

Open graytaylor0 opened this issue 4 months ago • 0 comments

Description

The geoip and dissect processors contained a Collection<Enum> and a Map<String, Enum> that the new enum deserializer was getting called for, and it does not deserialize on the types.

As a short term fix, add @JsonIgnore annotation the geoip enum getter method, and modify the dissect processor to use Map<String, String> with a custom AssertTrue validation and conversion of the target type from string to enum

Issues Resolved

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • [x] New functionality includes testing.
  • [ ] New functionality has a documentation issue. Please link to it in this PR.
    • [ ] New functionality has javadoc added
  • [x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.

graytaylor0 avatar Oct 25 '24 22:10 graytaylor0