fizzbeom opened a new pull request, #4421: URL: https://github.com/apache/logging-log4j2/pull/4421
`GelfLayout` builds its JSON output by hand using `JsonUtils.quoteAsString`, but the existing tests only covered plain values. - `JsonUtilsTest`: compare the output against Jackson's `JsonStringEncoder` for every Unicode code point, and check the common escape sequences. - `GelfLayoutTest`: log an event whose message, logger, thread, context data, exception and map message fields contain JSON special and delimiter characters, and check that the output parses back to the original values and that the NUL and LF delimiters appear only at the end. **INSERT HERE** a clear and concise description of what the pull request is for along with a reference to the associated issue IDs, if they exist. ## Checklist * Base your changes on `2.x` branch if you are targeting Log4j 2; use `main` otherwise * `./mvnw verify` succeeds ([the build instructions](https://logging.apache.org/log4j/2.x/development.html#building)) * Non-trivial changes contain an entry file in the `src/changelog/.2.x.x` directory * Tests are provided -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
