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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
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 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 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 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 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 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 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 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 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 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
https://ci.trafficserver.apache.org/job/docs-master/202/
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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
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 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 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 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 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 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 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
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 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
72 matches
Mail list logo