[GitHub] trafficserver issue #1370: Issue #1369: proxy.config.http.server_max_connect...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1370 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1393/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...

Github user persiaAziz commented on the issue: https://github.com/apache/trafficserver/pull/1357 I was thinking to have one overridable config for the client cert containing the partial or absolute path to avoid the memory allocation in Layout, but that will break the convention of

[GitHub] trafficserver pull request #1374: Fix CID 1369398: Null pointer dereferences

GitHub user randall opened a pull request: https://github.com/apache/trafficserver/pull/1374 Fix CID 1369398: Null pointer dereferences You can merge this pull request into a Git repository by running: $ git pull https://github.com/randall/trafficserver fix_1369398

[GitHub] trafficserver pull request #1368: Issue #1367: HdrHeap potential corruption

Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/1368 --- 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

[GitHub] trafficserver issue #1370: Issue #1369: proxy.config.http.server_max_connect...

Github user shinrich commented on the issue: https://github.com/apache/trafficserver/pull/1370 Ok. Still figuring out the new world order :-) --- 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

[GitHub] trafficserver issue #1366: Issue #1360: REGRESSION_TEST(SDK_API_OVERRIDABLE_...

Github user shinrich commented on the issue: https://github.com/apache/trafficserver/pull/1366 Probably. We will be pulling this back into our 7.1 --- 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

[GitHub] trafficserver pull request #1366: Issue #1360: REGRESSION_TEST(SDK_API_OVERR...

Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/1366 --- 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

[GitHub] trafficserver issue #1367: HdrHeap potential corruption

Github user shinrich commented on the issue: https://github.com/apache/trafficserver/issues/1367 Hopefully this fixes what you saw @maskit. Looking at the discussion on TS-2792 had it been solved before? --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1374: Fix CID 1369398: Null pointer dereferences

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1374 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1391/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver pull request #1357: TS-5022: reduce memory allocation in clien...

Github user SolidWallOfCode commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1357#discussion_r97822859 --- Diff: proxy/http/HttpSM.cc --- @@ -5050,9 +5036,26 @@ HttpSM::do_http_server_open(bool raw) opt.set_sni_servername(host, len);

[GitHub] trafficserver issue #1374: Fix CID 1369398: Null pointer dereferences

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1374 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1285/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1370: Issue #1369: proxy.config.http.server_max_...

Github user SolidWallOfCode commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1370#discussion_r97822530 --- Diff: proxy/http/HttpSM.cc --- @@ -4933,8 +4933,10 @@ HttpSM::do_http_server_open(bool raw) // between the statement above and

[GitHub] trafficserver issue #1303: TS-5107: Changes to autolinking in Docutils 1.3

Github user jsime commented on the issue: https://github.com/apache/trafficserver/pull/1303 The changes are fine and everything builds fine - I don't see any adverse effects from the inliner for Jira issues. I'm okay with merging this in for 7.1. I'm not sure we'd necessarily end up

[GitHub] trafficserver issue #1303: TS-5107: Changes to autolinking in Docutils 1.3

Github user jablko commented on the issue: https://github.com/apache/trafficserver/pull/1303 @zwoop Sure thing. --- 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

[GitHub] trafficserver issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1378 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1396/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1341: openssl errors for every request

Github user bryancall commented on the issue: https://github.com/apache/trafficserver/issues/1341 This has been resloved --- 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

[GitHub] trafficserver issue #1341: openssl errors for every request

Github user bryancall closed the issue at: https://github.com/apache/trafficserver/issues/1341 --- 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 issue #1344: Origin SNI

Github user bryancall commented on the issue: https://github.com/apache/trafficserver/issues/1344 @brkishore Only committers can have issues assigned to them. It is a limitation of GitHub. Please submit a PR for it once you have a fix. Thank you! --- If your project is

[GitHub] trafficserver issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1378 Also, please remember to add Milestone, Label's etc. on each Issue / PR, otherwise @bryancall gets all pissy. :) --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver pull request #1379: Make sure to schedule connect event on cor...

GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/1379 Make sure to schedule connect event on correct thread type. Cannot blindly schedule on current thread. It may not be the right type. You can merge this pull request into a Git repository

[GitHub] trafficserver issue #1379: Make sure to schedule connect event on correct th...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1379 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1397/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1377: TS-5107: init_customizations() isn't new in Docut...

Github user jablko commented on the issue: https://github.com/apache/trafficserver/pull/1377 @jsime Thank you! --- 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

[GitHub] trafficserver issue #1370: Issue #1369: proxy.config.http.server_max_connect...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1370 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1287/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1377: TS-5107: init_customizations() isn't new in Docut...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1377 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1289/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1378: 7.1.x TS-5107: Changes to autolinking in D...

GitHub user jablko opened a pull request: https://github.com/apache/trafficserver/pull/1378 7.1.x TS-5107: Changes to autolinking in Docutils 0.13 You can merge this pull request into a Git repository by running: $ git pull https://github.com/jablko/trafficserver 7.1.x

[GitHub] trafficserver issue #1377: TS-5107: init_customizations() isn't new in Docut...

Github user jsime commented on the issue: https://github.com/apache/trafficserver/pull/1377 @jablko As much my fault as anything on #1303 for not testing against the same package versions as the doc server. This PR builds fine for me now with sphinx 1.4.8 and docutils 0.12.0. Thanks

[GitHub] trafficserver pull request #1377: TS-5107: init_customizations() isn't new i...

Github user jsime closed the pull request at: https://github.com/apache/trafficserver/pull/1377 --- 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 issue #1375: Incorrectly freeing Http1ClientSession setting up...

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1375 Is this a 7.1.x candidate? --- 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

[GitHub] trafficserver issue #1368: Issue #1367: HdrHeap potential corruption

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1368 @shinrich Should we cherry-pick this back to 7.1.x ? --- 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

Re: Build failed in Jenkins: docs-master #201

I'll have a fix ready in a sec. Sorry about this! On 25/01/17 01:08 PM, Jon Sime wrote: Sure enough, if I downgrade Sphinx to 1.4.8 it's failing in that same spot for me. Looks like PR 1303 that I merged today was depending on changes from the stable 1.5 release. On Wed, Jan 25, 2017 at 1:53

[GitHub] trafficserver issue #1379: Make sure to schedule connect event on correct th...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1379 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1291/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1376: TSStringPercentDecode null-termination cli...

Github user shinrich closed the pull request at: https://github.com/apache/trafficserver/pull/1376 --- 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

[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...

Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/1357 I think I want a different approach on this. I'll talk with @persiaAziz. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] trafficserver pull request #1376: TSStringPercentDecode null-termination cli...

GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/1376 TSStringPercentDecode null-termination clipping on overflow. Actually fixed by @petar last summer, but forgot to get it pushed to up. Long URL's would cause buffer overflow write.

[GitHub] trafficserver issue #1376: TSStringPercentDecode null-termination clipping o...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1376 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1394/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1377: TS-5107: init_customizations() isn't new in Docut...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1377 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1395/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...

Github user SolidWallOfCode commented on the issue: https://github.com/apache/trafficserver/pull/1357 The problem (as @persiaAziz patiently explained) is the client certificate is overridable which means it cannot be reliable precomputed when `remap.config` is parsed. In theory a

[GitHub] trafficserver issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1378 Why are there 4 commits on this? Can you squash those? --- 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

[GitHub] trafficserver issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1378 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1290/ for details. --- If your project is set up for it, you can reply to this email and have your

Jenkins build is back to normal : docs-master #202

https://ci.trafficserver.apache.org/job/docs-master/202/

[GitHub] trafficserver pull request #1374: Fix CID 1369398: Null pointer dereferences

Github user zwoop closed the pull request at: https://github.com/apache/trafficserver/pull/1374 --- 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 issue #1303: TS-5107: Changes to autolinking in Docutils 1.3

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1303 This doesn't merge cleanly into 7.1.x branch (conflicts on the python script). @jablko Can you make a new PR, for the 7.1.x branch please? --- If your project is set up for it, you can reply

[GitHub] trafficserver issue #1376: TSStringPercentDecode null-termination clipping o...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1376 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1288/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1377: TS-5107: init_customizations() isn't new i...

GitHub user jablko opened a pull request: https://github.com/apache/trafficserver/pull/1377 TS-5107: init_customizations() isn't new in Docutils 0.13 We can use it in 0.12 and 0.13. @jsime Thanks for merging #1303. Sorry for the mistake! If you approve, can you please

[GitHub] trafficserver issue #1368: Issue #1367: HdrHeap potential corruption

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1368 @maskit @shinrich ping? Should we back port to 7.1.x ? --- 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

[GitHub] trafficserver pull request #1373: #1279 - CID 1367531: Uninitialized members...

Github user bryancall closed the pull request at: https://github.com/apache/trafficserver/pull/1373 --- 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

[GitHub] trafficserver issue #1380: Fixed issue where somaxconn was being set to 0 on...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1380 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1399/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user jablko commented on the issue: https://github.com/apache/trafficserver/pull/1378 > Also, please remember to add Milestone, Label's etc. on each Issue / PR, otherwise @bryancall gets all pissy. :) Will do! :-) > Why are there 4 commits on this? Can you

[GitHub] trafficserver issue #1380: Fixed issue where somaxconn was being set to 0 on...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1380 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1400/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1380: Fixed issue where somaxconn was being set to 0 on...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1380 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1294/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver issue #1275: TS-5105: Do vc->con.setRemote(target) before sock...

Github user bryancall commented on the issue: https://github.com/apache/trafficserver/pull/1275 Please make sure pull requests are reviewed before merging. --- 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

[GitHub] trafficserver issue #1275: TS-5105: Do vc->con.setRemote(target) before sock...

Github user oknet commented on the issue: https://github.com/apache/trafficserver/pull/1275 Got it. --- 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

[GitHub] trafficserver issue #1380: Fixed issue where somaxconn was being set to 0 on...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1380 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1401/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1283: CID 1367526: CID 1367525 : money_trace/money_trac...

Github user bryancall closed the issue at: https://github.com/apache/trafficserver/issues/1283 --- 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 issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1378 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1398/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1378: 7.1.x TS-5107: Changes to autolinking in Docutils...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1378 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1292/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1380: Fixed issue where somaxconn was being set ...

GitHub user bryancall opened a pull request: https://github.com/apache/trafficserver/pull/1380 Fixed issue where somaxconn was being set to 0 on OSX and preventing ATS to work You can merge this pull request into a Git repository by running: $ git pull

[GitHub] trafficserver issue #1380: Fixed issue where somaxconn was being set to 0 on...

Github user zwoop commented on the issue: https://github.com/apache/trafficserver/pull/1380 ink_inet.cc:613:10: error: unused variable 'value_size' [-Werror=unused-variable] size_t value_size = sizeof(value); --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver issue #1345: Crash on retry to open server connection with POS...

Github user oknet commented on the issue: https://github.com/apache/trafficserver/issues/1345 @bcall Sorry, I'm not familiar with HttpSM. Also I think this issue is covered in the topic that "Kees Spoelstra" described in secur...@trafficserver.apache.org. 2017-01-26

[GitHub] trafficserver issue #1279: CID 1367531: Uninitialized members (UNINIT_CTOR):...

Github user bryancall commented on the issue: https://github.com/apache/trafficserver/issues/1279 This was closed in PR #1279 --- 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

[GitHub] trafficserver issue #1279: CID 1367531: Uninitialized members (UNINIT_CTOR):...

Github user bryancall closed the issue at: https://github.com/apache/trafficserver/issues/1279 --- 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 issue #1380: Fixed issue where somaxconn was being set to 0 on...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1380 Linux build *failed*! See https://ci.trafficserver.apache.org/job/Github-Linux/1293/ for details. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request #1380: Fixed issue where somaxconn was being set ...

Github user bryancall closed the pull request at: https://github.com/apache/trafficserver/pull/1380 --- 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

Build failed in Jenkins: docs-master #201

https://ci.trafficserver.apache.org/job/docs-master/201/Changes: [Leif Hedstrom] Modify TSHttpTxnHookAdd to add hook only once [Leif Hedstrom] [TS-5024] Ran clang-format. Modify TSHttpTxnHookAdd to add hook only [Chao Xu] Do not reschedule the periodic inactive_event and cancel it after

[GitHub] trafficserver pull request #1303: TS-5107: Changes to autolinking in Docutil...

Github user jsime closed the pull request at: https://github.com/apache/trafficserver/pull/1303 --- 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 issue #1367: HdrHeap potential corruption

Github user maskit commented on the issue: https://github.com/apache/trafficserver/issues/1367 Yes, I solved the problem, but the coalescing issue is still alive. This kind of bugs can be made again until we get some measure for it. --- If your project is set up for it, you can

[GitHub] trafficserver pull request #1375: Incorrectly freeing Http1ClientSession set...

GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/1375 Incorrectly freeing Http1ClientSession setting up to return a error Saw a very deep stack. The following is the stop of the stack. {code} #0 0x in ?? () #1

[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1375 FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1392/ for details. --- If your project is set up for it, you can reply to this email and have

[GitHub] trafficserver issue #1375: Incorrectly freeing Http1ClientSession setting up...

Github user atsci commented on the issue: https://github.com/apache/trafficserver/pull/1375 Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1286/ for details. --- If your project is set up for it, you can reply to this email and have your

[GitHub] trafficserver pull request #1370: Issue #1369: proxy.config.http.server_max_...

Github user shinrich commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/1370#discussion_r97871781 --- Diff: proxy/http/HttpSM.cc --- @@ -4933,8 +4933,10 @@ HttpSM::do_http_server_open(bool raw) // between the statement above and the check

Re: Build failed in Jenkins: docs-master #201

Sure enough, if I downgrade Sphinx to 1.4.8 it's failing in that same spot for me. Looks like PR 1303 that I merged today was depending on changes from the stable 1.5 release. On Wed, Jan 25, 2017 at 1:53 PM, wrote: >

[GitHub] trafficserver issue #1370: Issue #1369: proxy.config.http.server_max_connect...

Github user shinrich commented on the issue: https://github.com/apache/trafficserver/pull/1370 Addressed Alan's comment. --- 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