On Thu, 2015-05-21 at 10:04 +0200, Bertrand Delacretaz wrote:
> Hi,
> 
> On Thu, May 21, 2015 at 8:03 AM, Ian Boston <[email protected]> wrote:
> > ...Putting this in Sling would benefit Sling and a wider community, 
> > +1 on
> > inclusion...
> 
> +1, under contrib/extensions for now I suppose.

+1

Only only took a brief look at the documentation in the pull request,
but I noticed the request headers docs, which mention the custom 'X-AEM
-Tracers' and 'X-AEM-Tracer-Config' headers. This should be chagend to
Sling-Tracers and Sling-Tracer-Config .

AEM -> Sling is self-explanatory,  and the X- prefix is deprecated with
RFC6648 [1]

Cheers,

Robert


[1]: http://tools.ietf.org/html/rfc6648

Reply via email to