Re: How to get clang-analyzer bugs?

2016-04-16 Thread Leif Hedstrom
It's on the left column on CI.trafficserver.apache.org. -- Leif > On Apr 16, 2016, at 12:05 PM, Susan Hinrichs > wrote: > > Hello, > > It looks like my commit introduced two bugs into scan-build. The output > shows > > scan-build: 2 bugs found. > scan-build: Run 'scan-view > /home/jenki

How to get clang-analyzer bugs?

2016-04-16 Thread Susan Hinrichs
Hello, It looks like my commit introduced two bugs into scan-build. The output shows scan-build: 2 bugs found. scan-build: Run 'scan-view /home/jenkins/clang-analyzer/2016-04-16-165815-27698-1' to examine bug reports. Build step 'Execute shell' marked build as failure But I cannot find the

Re: Deprecation of SSL v2/3

2016-04-16 Thread Phil Sorber
Ok, here is my final plan then. I am going to mark them all deprecated for 6.2.x. Then after branching I am going to remove all client <-> proxy support and ifdef out proxy <-> origin support for SSLv3. SSLv2 will be totally gone. Then add a configure option that reads something like --enable-depr

[GitHub] trafficserver pull request: TS-3909 Further trampoline crash fixes...

2016-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/575 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] trafficserver pull request: Ts 3612 - Restructuring client session...

2016-04-16 Thread maskit
Github user maskit commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/570#discussion_r59969775 --- Diff: proxy/ProxyClientSession.h --- @@ -101,6 +106,75 @@ class ProxyClientSession : public VConnection void do_api_callout(TSHttpHookID id);

[GitHub] trafficserver pull request: TS-4240: fix coredump issues with esi ...

2016-04-16 Thread shukitchan
Github user shukitchan commented on the pull request: https://github.com/apache/trafficserver/pull/503#issuecomment-210846959 @jpeach patch is updated. please take a look. I will test more tonight and if it is good, i will squash the commit before merge. --- If your project is

[GitHub] trafficserver pull request: TS-3909 Further trampoline crash fixes...

2016-04-16 Thread shinrich
Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/575#issuecomment-210828998 I think a semi-reliable regression test would be very difficult. This is fixing a timing based problem. You'd have to have a client open a connection. Send a

[GitHub] trafficserver pull request: Ts 3612 - Restructuring client session...

2016-04-16 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/570 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] trafficserver pull request: Ts 3612 - Restructuring client session...

2016-04-16 Thread shinrich
Github user shinrich commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/570#discussion_r59967433 --- Diff: proxy/ProxyClientSession.h --- @@ -101,6 +106,75 @@ class ProxyClientSession : public VConnection void do_api_callout(TSHttpHookID id)

[GitHub] trafficserver pull request: Ts 3612 - Restructuring client session...

2016-04-16 Thread maskit
Github user maskit commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/570#discussion_r59965426 --- Diff: proxy/ProxyClientSession.h --- @@ -101,6 +106,75 @@ class ProxyClientSession : public VConnection void do_api_callout(TSHttpHookID id);

[GitHub] trafficserver pull request: Ts 3612 - Restructuring client session...

2016-04-16 Thread maskit
Github user maskit commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/570#discussion_r59965264 --- Diff: proxy/ProxyClientSession.h --- @@ -25,14 +25,19 @@ #define __PROXY_CLIENT_SESSION_H__ #include "ts/ink_platform.h" +#includ