You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by maskit <gi...@git.apache.org> on 2017/02/02 00:22:25 UTC

[GitHub] trafficserver pull request #1409: Log crc field and psql field correctly wit...

GitHub user maskit opened a pull request:

    https://github.com/apache/trafficserver/pull/1409

    Log crc field and psql field correctly with H2

    The two field values were logged incorrectly if a resopnse body is short.
    
    Because a write VIO isn't pass to a consumer if whole response body was sent before calling `update_write_request `, the field values aren't collected. The flag `retval` is used for switching whether a VIO will be passed.
    
    Also, 6.2.x has the same problem.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/maskit/trafficserver h2_logging

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/trafficserver/pull/1409.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 #1409
    
----
commit b048dcd81d066f0fcb8b2f8b80fe25aa0ca164db
Author: Masakazu Kitajo <ma...@apache.org>
Date:   2017-02-01T23:54:52Z

    Log crc field and psql field correctly with H2
    
    The two field values were logged incorrectly if a resopnse body is short.

----


---
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 #1409: Log crc field and psql field correctly with H2

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1409
  
    FreeBSD build *successful*! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/1453/ 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 #1409: Log crc field and psql field correctly wit...

Posted by zwoop <gi...@git.apache.org>.
Github user zwoop closed the pull request at:

    https://github.com/apache/trafficserver/pull/1409


---
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 #1409: Log crc field and psql field correctly with H2

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1409
  
    Linux build *successful*! See https://ci.trafficserver.apache.org/job/Github-Linux/1346/ 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 #1409: Log crc field and psql field correctly with H2

Posted by atsci <gi...@git.apache.org>.
Github user atsci commented on the issue:

    https://github.com/apache/trafficserver/pull/1409
  
    clang-analyzer build *successful*! See https://ci.trafficserver.apache.org/job/Github-Clang/18/ 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.
---