brbzull0 commented on code in PR #13609: URL: https://github.com/apache/trafficserver/pull/13609#discussion_r3905909337
########## doc/developer-guide/jsonrpc/jsonrpc-architecture.en.rst: ########## @@ -74,6 +74,10 @@ Our JSONRPC protocol implementation uses lib yamlcpp for parsing incoming and o this allows the server to accept either JSON or YAML format messages which then will be parsed by the protocol implementation. This seems handy for user that want to feed |TS| with existing yaml configuration without the need to translate yaml into json. +Null values on the way out are emitted as literal ``null`` rather than yaml's ``~``, as ``~`` is not accepted by JSON parsers. Both +spellings resolve to null when read as yaml, so this does not affect messages that are consumed as yaml, nor the ability to send +yaml to the server. Review Comment: Addressed in f6ee3fb254 — the paragraph now capitalizes `YAML` throughout and is rewritten in active voice. -- 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]
