Uploaded image for project: 'Confluence Data Center'
  1. Confluence Data Center
  2. CONFSERVER-58469

Make EnrichableMap deserialization opt-in

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Fixed
    • 5.6
    • None
    • We collect Confluence feedback from various sources, and we evaluate what we've collected when planning our product roadmap. To understand how this piece of feedback will be reviewed, see our Implementation of New Features Policy.

    Description

      Derived from CONFDEV-24631 and some PR comments, not all Maps should be converted to EnrichableMap when deserialized from JSON.

      Opting in can be done with the @JsonDeserialize(as=EnrichableMap.class) annotation, but there's a catch.

      A spike of implementing this shows that it breaks Content+Space roundtrip unit tests that can't get the EnrichedMapDeserializer from the SerializerModule because it's in conf-rest-serialization module while the unit tests are in conf-java-api.

      If we move all of the roundtrip tests to the rest module, we should be able to keep the java-api test code very lean.

      Attachments

        Activity

          People

            dtaylor David Taylor (Inactive)
            dtaylor David Taylor (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: