[GitHub] trafficserver pull request: TS-3948 Lock g_records during in RecDu...

2015-10-08 Thread sekimura
Github user sekimura commented on the pull request: https://github.com/apache/trafficserver/pull/304#issuecomment-146752220 @jpeach I've added backtrace when we got a crash https://issues.apache.org/jira/browse/TS-3948 --- If your project is set up for it, you can reply to this email

[GitHub] trafficserver pull request: TS-3948 Lock g_records during in RecDu...

2015-10-08 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/304#issuecomment-146751892 How does this happen? There are lots of places that iterate over the records without holding the read lock. We should make sure that the lock is held correctly and

[GitHub] trafficserver pull request: TS-3948 Lock g_records during in RecDu...

2015-10-08 Thread sekimura
GitHub user sekimura opened a pull request: https://github.com/apache/trafficserver/pull/304 TS-3948 Lock g_records during in RecDumpRecords to avoid a race Under certain conditions, data passing to RecDumpEntryCb and data inside the callback can be different. The problem is that g_

[GitHub] trafficserver pull request: OCSP docs

2015-10-08 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/303 --- 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-3961) open source Yahoo's ats-mult...

2015-10-08 Thread bgaff
Github user bgaff commented on the pull request: https://github.com/apache/trafficserver/pull/302#issuecomment-146719893 @dmorilha , thanks! --- 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 th

[GitHub] trafficserver pull request: OCSP docs

2015-10-08 Thread AZNova
GitHub user AZNova opened a pull request: https://github.com/apache/trafficserver/pull/303 OCSP docs Added proxy.config.ssl.ocsp.* configuration directives to records.config doc. Also added a brief OCSP Stapling section to Administrator's Guide. You can merge this pull request int

[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...

2015-10-08 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/302#issuecomment-146685122 Great, thanks! --- 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 f

[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...

2015-10-08 Thread dmorilha
Github user dmorilha commented on the pull request: https://github.com/apache/trafficserver/pull/302#issuecomment-146684662 @zwoop will work on these items later, thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...

2015-10-08 Thread zwoop
Github user zwoop commented on the pull request: https://github.com/apache/trafficserver/pull/302#issuecomment-146684009 This also must be run through clang-format first. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...

2015-10-08 Thread PSUdaemon
Github user PSUdaemon commented on the pull request: https://github.com/apache/trafficserver/pull/302#issuecomment-146683942 I should also add this looks very interesting and thanks for contributing it! --- If your project is set up for it, you can reply to this email and have your r

[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...

2015-10-08 Thread PSUdaemon
Github user PSUdaemon commented on the pull request: https://github.com/apache/trafficserver/pull/302#issuecomment-146682956 This belongs under the experimental directory. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. I

[GitHub] trafficserver pull request: (TS-3961) open source Yahoo's ats-mult...

2015-10-08 Thread dmorilha
GitHub user dmorilha opened a pull request: https://github.com/apache/trafficserver/pull/302 (TS-3961) open source Yahoo's ats-multiplexer plug-in You can merge this pull request into a Git repository by running: $ git pull https://github.com/dmorilha/trafficserver multiplexer