bryancall opened a new pull request, #12947:
URL: https://github.com/apache/trafficserver/pull/12947

   ## Summary
   
   * **Add `Content-Type` directive to `.body_factory_info`** -- Operators can 
now set the MIME type for body factory error responses (e.g. `Content-Type: 
text/plain`) instead of the hardcoded `text/html` default. The directive is 
parsed alongside the existing `Content-Language` and `Content-Charset` 
directives.
   
   * **Fix Content-Type overwrite in `HttpSM::setup_internal_transfer`** -- The 
internal transfer setup was unconditionally resetting `Content-Type` to 
`text/html` when `internal_msg_buffer_type` was NULL, masking any type set by 
the body factory. Now uses the `MIME_PRESENCE_CONTENT_TYPE` presence bit to 
only apply the default when Content-Type is not already present.
   
   * **Add documentation for `.body_factory_info` metadata file** -- The admin 
guide's error messages page now documents all three supported directives 
(`Content-Language`, `Content-Charset`, `Content-Type`) with examples.
   
   * **Add autest** -- New `body_factory_content_type.test.py` verifies both 
default (`text/html`) and custom (`text/plain`) Content-Type behavior.
   
   ## Testing
   
   - Built with ASAN, tested manually on eris (default and custom Content-Type 
verified via curl)
   - Autest passes: both default and custom Content-Type scenarios
   
   Fixes: #10893
   


-- 
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]

Reply via email to