Error when generating/serializing keys with multilines and colon

Hello there

I wanted to report that a key containing a colon : followed by new line chars \n does not seem to be parsing correctly. For example the following test case reproduces the error

    @Test(expected = JsonParseException.class)
    public void testYamlMultiLineNotCanonical() throws Exception {
        String key = "SomeKey:\n\nOtherLine";
        ObjectMapper jsonMapper = new ObjectMapper();
        ObjectNode jsonRoot = jsonMapper.createObjectNode();
        jsonRoot.put(key, 302);

        YAMLFactory yamlFactory = new YAMLFactory();
        ObjectMapper yamlObjectMapper = new ObjectMapper(yamlFactory);
        yamlObjectMapper.configure(JsonParser.Feature.STRICT_DUPLICATE_DETECTION, true);
        StringWriter swriter = new StringWriter();

        yamlObjectMapper.writeValue(swriter, jsonRoot);

        ObjectMapper readMapper =  new ObjectMapper(yamlFactory);
        ObjectNode readJsonRoot = (ObjectNode) readMapper.readTree(swriter.toString());
        Assert.assertEquals(readJsonRoot.get(key).asInt(),302 );
    }

I would have expected the parser to quote the key as \n and : are supported yaml content.

There is work around for this: Force canonical output.

        yamlFactory.configure(YAMLGenerator.Feature.CANONICAL_OUTPUT, true);

That however could have other side effects. Maybe I am missing some other configuration that would do the same?

Thoughts, comments are welcomed.

[cowtowncoder]

I added a simple failing test like so:

    public void testComplexName() throws Exception
    {
        final String key = "SomeKey:\nOtherLine";
        Map<?,?> input = Collections.singletonMap(key, 302);
        final String doc = MAPPER.writeValueAsString(input);
        Map<?,?> actual = MAPPER.readValue(doc, Map.class);
        assertEquals(input, actual);
    }

and will try to see if I can figure out how to resolve this, since SnakeYAML should be able to handle it properly.

[cowtowncoder]

@asomov Will do if need be, thanks!

Fix is in, for 2.13.2 release. Covers linefeed case but I suspect there may be other things that require quoting.