You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by SolidWallOfCode <gi...@git.apache.org> on 2017/02/06 22:00:34 UTC
[GitHub] trafficserver pull request #1422: YTS-614: Avoid crash due to error during P...
GitHub user SolidWallOfCode opened a pull request:
https://github.com/apache/trafficserver/pull/1422
YTS-614: Avoid crash due to error during POST request transform.
If the user agent has an error during a POST transform ATS can either crash (by trying to write an error response to a dead user agent) or leave the request in an undead state waiting on events that won't happen.
(cherry picked from commit fad1f6ed353d332ce505306f462e9f90be00a6ad)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/SolidWallOfCode/trafficserver yts-614
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/trafficserver/pull/1422.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1422
----
commit dee344ca1f02d1532c02abf2b2333f9091b4955f
Author: Alan M. Carroll <so...@yahoo-inc.com>
Date: 2015-12-15T17:08:53Z
YTS-614: Avoid crash due to error during POST request transform.
(cherry picked from commit fad1f6ed353d332ce505306f462e9f90be00a6ad)
----
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by SolidWallOfCode <gi...@git.apache.org>.
Github user SolidWallOfCode commented on the issue:
https://github.com/apache/trafficserver/pull/1422
We've been running this in production for months and it fixed a series of crashes in ATS for us. I think it should probably be backported to 7.1, Y! will be putting it in our 7.1 fork.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1422
clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/44/ for details.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1422
FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1481/ for details.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1422
Linux build *successful*! See https://ci.trafficserver.apache.org/job/linux-github/1374/ for details.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1422
clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/clang-analyzer-github/45/ for details.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1422
FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/freebsd-github/1480/ for details.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver pull request #1422: YTS-614: Avoid crash due to error during P...
Posted by SolidWallOfCode <gi...@git.apache.org>.
Github user SolidWallOfCode closed the pull request at:
https://github.com/apache/trafficserver/pull/1422
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by zwoop <gi...@git.apache.org>.
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1422
@SolidWallOfCode Is this something you feel comfortable with, and should be a 7.1.x back port?
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] trafficserver issue #1422: YTS-614: Avoid crash due to error during POST req...
Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1422
Linux build *successful*! See https://ci.trafficserver.apache.org/job/linux-github/1373/ for details.
---
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 is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---