mlibbey commented on code in PR #12174: URL: https://github.com/apache/trafficserver/pull/12174#discussion_r2036196101
########## doc/admin-guide/plugins/header_rewrite.en.rst: ########## @@ -1541,8 +1545,17 @@ could each be tagged with a consistent name to make finding logs easier.:: sometimes avoid setting internal ``@`` headers for passing information between hooks. These internal state variables are much more efficient than setting and reading headers. +Prepend a Path to the URL +------------------------- + +The following prepends ``/foo/bar`` to the URL path. Note that since ``PATH`` is +non-inclusive of the initial ``/``, the provided prefix does not start with a ``/``:: Review Comment: Like the doc updates! Interestingly, this week we were troubleshooting a string concat problem, where I believe the solution was to remove the quotes (they yielded a Warning). The initial string was like set-header X-Foo "%{HEADER:Cache-Control}, %{Age}" ... the warning went away with set-header X-Foo %{HEADER:Cache-Control}, %{Age} -- 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-unsubscr...@trafficserver.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org