[GitHub] trafficserver pull request: TS-3649 Fix for url_sig plugin securit...

2015-06-01 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/208 TS-3649 Fix for url_sig plugin security issues (crash by HTTP request & circumvent signature). You can merge this pull request into a Git repository by running: $ git pull h

[GitHub] trafficserver pull request: TS-3740: header_rewrite set-redirect e...

2015-07-15 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/253 TS-3740: header_rewrite set-redirect enhancement. header_rewrite set-redirect support when not called from remap plugin. You can merge this pull request into a Git repository by running

[GitHub] trafficserver pull request: [ts-4023] cachekey plugin

2015-12-09 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/371 [ts-4023] cachekey plugin This plugin allows some common cache key manipulations based on various HTTP request elements. It can - sort query parameters so reordering can be a cache hit

[GitHub] trafficserver pull request: TS-4176: s3_auth add support for matri...

2016-02-17 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/455#issuecomment-185330773 - Reviewed and tested the patch and it looks OK. - Verified that it does not break backward compatibility, tested end-to-end against S3 with both virtual

[GitHub] trafficserver pull request: TS-4161: fixed ProcessManager stackove...

2016-02-23 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/496 TS-4161: fixed ProcessManager stackoverflow vulnerability You can merge this pull request into a Git repository by running: $ git pull https://github.com/gtenev/trafficserver TS-4161

[GitHub] trafficserver pull request: TS-4161: fixed ProcessManager stackove...

2016-03-04 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/496#issuecomment-192317642 Sure, will remove the `malloc` failure check. We could increase the limit much higher or remove it altogether. With the previous code in the really

[GitHub] trafficserver pull request: TS-4161: fixed ProcessManager stackove...

2016-03-06 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/496#issuecomment-193087594 Removed `malloc` failure check as recommended and increased the limit. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request: TS-2642 Adds a new operator %{NOW} to ...

2016-03-07 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/517#issuecomment-193159857 +1 --- 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

[GitHub] trafficserver pull request: TS-4161 ProcessManager::pollLMConnecti...

2016-03-07 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/519 TS-4161 ProcessManager::pollLMConnection code refactoring Converted while(1) to for(;;) loop for better readability, suggested by TS-4161 previous pull request comments. You can merge

[GitHub] trafficserver pull request: TS-4183: cachekey: URI/prefix/path cap...

2016-04-06 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/552 TS-4183: cachekey: URI/prefix/path capture/replacement --capture-prefix-uri=regex --capture-prefix-uri=/regex/replacement/ --capture-path-uri=regex --capture-path-uri=/regex

[GitHub] trafficserver pull request: TS-4147 Allow gzip plugin to be a rema...

2016-04-18 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/579 TS-4147 Allow gzip plugin to be a remap plugin pull request on behalf of @meeramn You can merge this pull request into a Git repository by running: $ git pull https://github.com/gtenev

[GitHub] trafficserver pull request: TS-4356 Deprecate cacheurl plugin

2016-04-18 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/582 TS-4356 Deprecate cacheurl plugin Added a "deprecated" message and `cachekey` examples to the `cacheurl` doc. You can merge this pull request into a Git repository by running:

[GitHub] trafficserver pull request: TS-4356 Deprecate cacheurl plugin

2016-04-18 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/582#issuecomment-211700790 Sounds good, will add the message. --- 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

[GitHub] trafficserver pull request: TS-4356 Deprecate cacheurl plugin

2016-05-06 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/582#issuecomment-217512682 PR updated, TS-4356 already marked "Backport to Version: 6.2.0" --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] trafficserver pull request: TS-4020 Use cache-key URL for some plu...

2016-05-06 Thread gtenev
Github user gtenev commented on the pull request: https://github.com/apache/trafficserver/pull/619#issuecomment-217514025 +1 --- 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

[GitHub] trafficserver pull request: TS-4366 Uninitialized stack value used...

2016-05-22 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/656 TS-4366 Uninitialized stack value used in mp4 plugin It is possible that there are cases where IOBufferReaderCopy() does not modify the input buffer (copy 0 bytes) which leaves the buffer

[GitHub] trafficserver pull request: TS-3340: Coverity fixes: CID #1022050

2015-01-30 Thread gtenev
GitHub user gtenev opened a pull request: https://github.com/apache/trafficserver/pull/167 TS-3340: Coverity fixes: CID #1022050 Fixed unchecked-return-value issues in the build number generation. Validated input date and time format and generated a fixed-size build number