Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/697
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/288/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/697
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/180/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply app
Github user shukitchan commented on the issue:
https://github.com/apache/trafficserver/pull/697
@jpeach , I just updated according to your comments. pls take a look.
once i get a thumbs up, will definitely squash the commits before merge
---
If your project is set up for it, you c
Github user zizhong commented on the issue:
https://github.com/apache/trafficserver/pull/692
Thanks for the instructive comments. I've fixed most of them. For the
namespace, I would keep it as it makes code clearer.
About supporting **Set-Cookie**, let me address it with another ti
Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/644
OK, I will do, but could you first look at the first two commits separately?
There are various operation logic and I have some concerns about it.
At first one I calculate time which
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/697
This looks OK to me, though it is a little opaque in the way that Lua stack
code tends to be. I would like to see some reduction in copy/paste in another
commit on this PR, but if you want to d
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/697#discussion_r67285947
--- Diff: plugins/experimental/ts_lua/ts_lua_server_request.c ---
@@ -258,10 +286,27 @@ ts_lua_server_request_get_headers(lua_State *L)
while (fie
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/697#discussion_r67285081
--- Diff: plugins/experimental/ts_lua/ts_lua_server_response.c ---
@@ -102,11 +104,27 @@ ts_lua_server_response_get_headers(lua_State *L)
while (f
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/697#discussion_r67285016
--- Diff: plugins/experimental/ts_lua/ts_lua_server_response.c ---
@@ -152,10 +172,21 @@ ts_lua_server_response_header_get(lua_State *L)
if (key &
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/697#discussion_r67284564
--- Diff: plugins/experimental/ts_lua/ts_lua_server_response.c ---
@@ -198,13 +230,25 @@ ts_lua_server_response_header_set(lua_State *L)
field_loc
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/666
@jsime Ready to merge this one?
---
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
enable
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/704
---
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 asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/718
---
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 yatsukhnenko closed the pull request at:
https://github.com/apache/trafficserver/pull/700
---
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 fe
Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/700
@oknet, no, this isn't helping.
It looks like cache write consumer tries to complete reading data from
transformation but server_session already closed because transformation reads
da
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/644
Sorry for the delay. Would you mind squashing this down to just one commit
as well please?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/644
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/287/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/644
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/179/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/644
[approve ci] Doing a fresh build.
---
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
enabl
Github user yatsukhnenko commented on the issue:
https://github.com/apache/trafficserver/pull/644
@zwoop, could you have a time to look closer at this PR?
---
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 do
This vote has failed. We will address a number of bugs that have been
identified as blockers and prepare a new RC.
Thanks.
On Tue, Jun 14, 2016 at 10:35 AM Bryan Call wrote:
> -1 - Running this release in production I am seeing ASAN issues and there
> is a problem with lots of error messages go
Github user shinrich closed the pull request at:
https://github.com/apache/trafficserver/pull/689
---
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 featur
Github user shinrich commented on the issue:
https://github.com/apache/trafficserver/pull/689
Closing. Turns out the issue was due to openssl version change. See Jira
comments for details.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user asfgit closed the pull request at:
https://github.com/apache/trafficserver/pull/703
---
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 closed the pull request at:
https://github.com/apache/trafficserver/pull/711
---
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 featu
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/692
Ok, so I have reviewed this, and I'm happy with it. At the risk of bike
shedding, I'm throwing in some thoughts on coding style here. It's entirely up
to you if you feel it worth changing or not
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/718
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/178/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/718
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/286/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/718
TS-4543: Fix core in HttpSM::tunnel_handler_post.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/shinrich/trafficserver ts-4543
Altern
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/717
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/285/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/717
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/177/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user shinrich opened a pull request:
https://github.com/apache/trafficserver/pull/717
TS-4507: Fixes to ensure SSN_CLOSE called after TXN_CLOSE.
Adding logic to ensure that SSN_CLOSE doesn't get called before TXN_CLOSE.
This involved deferring the session destroy further. Ha
32 matches
Mail list logo
