Binary body in plugin of traffic server

2015-07-06 Thread Jiří Podhorský
Hello, can be binary body of http response accessed from some plugin of ATS, for example jpg picture? I tried to look for it in TransformationPlugin https://docs.trafficserver.apache.org/en/latest/api/classatscppapi_1_1TransformationPlugin.html, but there is only sentence about producing binary

[GitHub] trafficserver pull request: TS-3742: ATS incorrectly advertises TL...

2015-07-06 Thread bgaff
Github user bgaff commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/241#discussion_r34001015 --- Diff: iocore/net/SSLUtils.cc --- @@ -308,8 +308,10 @@ set_context_cert(SSL *ssl) if (ctx != NULL) { SSL_set_SSL_CTX(ssl, ctx); #i

[GitHub] trafficserver pull request: add init test for SSL Termination

2015-07-06 Thread jacksontj
Github user jacksontj commented on the pull request: https://github.com/apache/trafficserver/pull/192#issuecomment-119040683 We've been doing that for all of the tests actually. In my tests at least I'm not verifying the validity of the cert-- since I can check to make sure its

[GitHub] trafficserver pull request: TS-3742: ATS incorrectly advertises TL...

2015-07-06 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/241 TS-3742: ATS incorrectly advertises TLS ticket extension. You can merge this pull request into a Git repository by running: $ git pull https://github.com/shinrich/trafficserver ts-3742

[GitHub] trafficserver pull request: Maintain and use a mapping of hostname...

2015-07-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/240 --- 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: add init test for SSL Termination

2015-07-06 Thread repodude
Github user repodude commented on the pull request: https://github.com/apache/trafficserver/pull/192#issuecomment-118970165 Hey Feifei, I'm curious why are we incorporating a static pre-generated certificate, rather than, perhaps a script that just generates one to be used for t

Re: [GitHub] trafficserver pull request: Maintain and use a mapping of hostname...

2015-07-06 Thread Brian Geffon
I agree that this is the preferred pattern given this situation, the message is very clear: this object is no longer in use and then when it's assigned again at the start it's clear it's in use again. On Mon, Jul 6, 2015 at 9:51 PM, SolidWallOfCode wrote: > Github user SolidWallOfCode commented

[GitHub] trafficserver pull request: Maintain and use a mapping of hostname...

2015-07-06 Thread SolidWallOfCode
Github user SolidWallOfCode commented on a diff in the pull request: https://github.com/apache/trafficserver/pull/240#discussion_r33935287 --- Diff: iocore/hostdb/HostDB.cc --- @@ -2833,15 +2650,8 @@ ParseHostFile(char const *path) } } - if (!HostFilePairs