[GitHub] trafficserver pull request: [TS-3476] Add a log tag for applicatio...

2015-07-30 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/250 --- 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 pull request: [TS-3476] Add a log tag for applicatio...

2015-07-13 Thread ericcarlschwartz
GitHub user ericcarlschwartz opened a pull request: https://github.com/apache/trafficserver/pull/250 [TS-3476] Add a log tag for application protocol (add docs) Needed to add to the docs for TS-3476 You can merge this pull request into a Git repository by running: $ git pull ht

[GitHub] trafficserver pull request: [TS-3476] Add a log tag for applicatio...

2015-07-10 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/244 --- 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 pull request: [TS-3476] Add a log tag for applicatio...

2015-07-09 Thread ericcarlschwartz
Github user ericcarlschwartz commented on the pull request: https://github.com/apache/trafficserver/pull/244#issuecomment-120004282 @shinrich ah yeah I wasn't sure why it'd been dropped but I also can't see any reason adding it back would introduce any problems either --- If your pro

[GitHub] trafficserver pull request: [TS-3476] Add a log tag for applicatio...

2015-07-09 Thread SolidWallOfCode
Github user SolidWallOfCode commented on the pull request: https://github.com/apache/trafficserver/pull/244#issuecomment-119953440 Looks reasonable as a stopgap to me. Hopefully we can in the near future do better through restructuring the underlying protocol handling structure as in

[GitHub] trafficserver pull request: [TS-3476] Add a log tag for applicatio...

2015-07-09 Thread shinrich
Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/244#issuecomment-119948800 Looks good to me. My only question was why had the PluginIdentity been earlier removed as a parent of the SpdyClientSession? @SolidWallOfCode dug up

[GitHub] trafficserver pull request: [TS-3476] Add a log tag for applicatio...

2015-07-08 Thread ericcarlschwartz
GitHub user ericcarlschwartz opened a pull request: https://github.com/apache/trafficserver/pull/244 [TS-3476] Add a log tag for application protocol You can merge this pull request into a Git repository by running: $ git pull https://github.com/yahoo/trafficserver TS-3476 Al