[GitHub] trafficserver pull request: Added origin resp time to access log

2014-10-08 Thread acaciocenteno
Github user acaciocenteno commented on the pull request: https://github.com/apache/trafficserver/pull/124#issuecomment-58430258 I updated the implementation to include the fields that Leif suggested and wrote the doc. Let me know if you guys need something else. --- If your project i

Re: How to properly log a string on access.log

2014-10-08 Thread Acácio Centeno
37. This length is fixed. I've looked marshal_str's code and my understanding is that it will pad the string with '$' to fill the necessary 40, right? so what gets copied to buf is actually something like "cdc19cac-527e-4e4b-b8f9-ef453db9e0d3$$$". Also, I'm not sure why the fields should use LogA

Re: How to properly log a string on access.log

2014-10-08 Thread James Peach
On Oct 8, 2014, at 11:43 AM, Acácio Centeno wrote: > Hello, > > I've written the following function to log our internal UUID to access.log: > > int > LogAccessHttp::marshal_proxy_uuid(char *buf) > { > const char *str = NULL; > int len = INK_MIN_ALIGN; > > if (buf) { >if ( m_http_sm->mag

Re: How to properly log a string on access.log

2014-10-08 Thread Acácio Centeno
Yes, it's guaranteed. And it's logging correctly on debug.log. Acácio Centeno Software Engineering Azion Technologies Porto Alegre, Brasil +55 51 3012 3005 | +55 51 8118 9947 Miami, USA +1 305 704 8816 Quaisquer informações contidas neste e-mail e anexos podem ser confidenciais e privilegiadas,

Re: How to properly log a string on access.log

2014-10-08 Thread Sudheer Vinukonda
Hi Acacio, Is m_http_sm->get_uuid() guaranteed to be null terminated? Thanks, Sudheer On 10/8/14, 11:43 AM, "Acácio Centeno" wrote: >Hello, > >I've written the following function to log our internal UUID to >access.log: > >int >LogAccessHttp::marshal_proxy_uuid(char *buf) >{ > const char *st

How to properly log a string on access.log

2014-10-08 Thread Acácio Centeno
Hello, I've written the following function to log our internal UUID to access.log: int LogAccessHttp::marshal_proxy_uuid(char *buf) { const char *str = NULL; int len = INK_MIN_ALIGN; if (buf) { if ( m_http_sm->magic == HTTP_SM_MAGIC_ALIVE ) { str = m_http_sm->get_uuid(); } el

Re: git commit: [TS-3121] Add method is_valid_response()

2014-10-08 Thread James Peach
On Oct 8, 2014, at 9:42 AM, sudhe...@apache.org wrote: > Repository: trafficserver > Updated Branches: > refs/heads/master a33073505 -> 6548088ec > > > [TS-3121] Add method is_valid_response() > > > Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo > Commit: http://git-wip-us

RE: localhost access

2014-10-08 Thread Luca Rea
Hi, Don't work, it seems that 127.0.0.1 skips the parent.config rules (bug or feature?), other suggestions? -Original Message- From: Leif Hedstrom [mailto:zw...@apache.org] Sent: mercoledì 8 ottobre 2014 17:53 To: dev@trafficserver.apache.org Cc: Alan Carroll Subject: Re: localhost acce

Re: localhost access

2014-10-08 Thread Leif Hedstrom
On Oct 8, 2014, at 9:05 AM, Luca Rea wrote: > Hi, > > Oct 8 16:47:25.042720 traffic_manager {0x7fdda89227e0} ERROR: (last system > error 32: Broken pipe) > Oct 8 16:47:25.078427 traffic_cop cop received child status signal [6816 > 256] > Oct 8 16:47:25.078458 traffic_cop traffic_manage

RE: localhost access

2014-10-08 Thread Luca Rea
Hi, I've removed the rules from iptables because they cause a restart loop: Oct 8 16:43:46.004807 traffic_cop (test) write failed [110 'Connection timed out'] Oct 8 16:43:46.004839 traffic_cop server heartbeat failed [1] Oct 8 16:44:59.005037 traffic_cop (test) write failed [110 'Connectio