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

Reply via email to