Re: [PR] Fix overridable configuration testing [trafficserver]

["bneradt (via GitHub)" <gi...@apache.org>]

bneradt commented on PR #13516:
URL: https://github.com/apache/trafficserver/pull/13516#issuecomment-5499872895

   Addressed the bounded SSL string issue in `7a25aba60d`.
   
   The direct SSL override setters now copy explicit-length values into stable `ParsedConfigCache` storage, which supplies the NUL terminator required by downstream C-string consumers. The transaction config also retains each supplied length, and the getters return that length instead of calling `strlen`.
   
   Added focused coverage that round-trips all eight direct SSL keys using a non-NUL-terminated three-byte value containing an embedded NUL. Verification in `asfats5`: full build/install passed, and `test_api` passed 1,506 assertions across 4 test cases.


-- 
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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org