Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69248777
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69248675
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69248581
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69248550
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69248235
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69248038
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user bgaff commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/776#discussion_r69247795
--- Diff: plugins/experimental/brotli_transform/brotli_transform.cc ---
@@ -0,0 +1,248 @@
+/**
+ Licensed to the Apache Software Foundation (ASF)
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/765
Yeah, maybe we should close this PR ?
---
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
e
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/770
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/293/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/770
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/397/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user zhouxiangxiang commented on the issue:
https://github.com/apache/trafficserver/pull/765
@zwoop I'm a newbie of ats.
I use it in my company, and the memory is always increasing.
The memory's management puzzle me a lot.
---
If your project is set up for it
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/770
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/292/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/770
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/396/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user zwoop closed the pull request at:
https://github.com/apache/trafficserver/pull/778
---
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 i
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/778
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/291/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user zwoop closed the pull request at:
https://github.com/apache/trafficserver/pull/747
---
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 i
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/778
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/395/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/778
TS-4625 epic: Adds a mutex to the continutation
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zwoop/trafficserver TS-4625
Alternatively
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/775
It seems there are merge conflicts here though, Tyler, can you please
rebase your tree (which probably means pushing up the main master into your
fork first) ?
---
If your project is set up fo
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/775
Yeah, this seems right, TSEventDestroy() checks its argument, and we use it
like this in several other places (which is likely why coverity gets confused).
Consistency is good!
---
If your pro
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/776
I'll have to review this properly, but an initial thought is that maybe
this should be part of the gzip plugin? Maybe that plugin gets renamed to
"compression" or something? It has plumbing for
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/770
Unless there's a penalty (or error) to always use $(sort ...), I'd
recommend doing so consistently. That avoids future kids from having to know
when, or when not to, use it.
Where is ti
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/290/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/394/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/747
wth, how did FreeBSD core dump ... [approve ci]
---
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
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/289/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/777
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/288/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/777
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/392/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/393/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/777
Need input from the git expert, too bad @bgaff doesn't have a computer
still :-/. So will have to settle for @jpeach and @PSUdaemon .
---
If your project is set up for it, you can reply to this
GitHub user zwoop opened a pull request:
https://github.com/apache/trafficserver/pull/777
TS-4623 Adds a pre-commit script for git
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/zwoop/trafficserver TS-4623
Alternatively you can
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/287/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/740
---
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 user jpeach closed the pull request at:
https://github.com/apache/trafficserver/pull/741
---
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 user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/391/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/775
s/NetworkUtilisRemote/NetworkUtilsRemote/ ;)
---
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 user shukitchan commented on the issue:
https://github.com/apache/trafficserver/pull/776
@bgaff i know you may have a plugin ready as well.
Anyway, we can either add our needs onto your code. Or you can start with
reviewing and merging this PR and add on your requirements o
GitHub user caricaturecm opened a pull request:
https://github.com/apache/trafficserver/pull/776
TS-4553: Brotli plugin
@bgaff can you please take a look? thanks.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/caricaturecm/traffi
Hello All,
I've prepared a release for v6.2.0 (RC2) which is the first stable release
in the 6.2.x series. This is also the beginning of our Long Term Support
(LTS) version for 6.x as detailed in our Release Management document:
https://cwiki.apache.org/confluence/display/TS/Release+Management
C
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69205590
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -202,34 +202,38 @@ handle_purge(TSHttpTxn txnp, PurgeInstance *purge)
bool should_p
Github user jacksontj closed the pull request at:
https://github.com/apache/trafficserver/pull/774
---
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 featu
GitHub user strotyl opened a pull request:
https://github.com/apache/trafficserver/pull/775
TS-4598 : Coverity Null-Check after deref in NetworkUtilisRemote.cc
This is fixing Coverity issue CID 1237320.
You can merge this pull request into a Git repository by running:
$ git pul
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/773
@jacksontj Maybe the right place to do this is when
``HttpSM::set_next_state()
`` handles ``SM_ACTION_DNS_LOOKUP``?
---
If your project is set up for it, you can reply to this email and ha
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69190550
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -143,18 +144,17 @@ on_http_cache_lookup_complete(TSHttpTxn txnp, TSCont
contp, PurgeInst
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69190382
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -202,34 +202,38 @@ handle_purge(TSHttpTxn txnp, PurgeInstance *purge)
bool should_
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/286/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/285/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/390/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/773
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/284/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/747
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/389/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/773
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/388/ for details.
---
If your project is set up for it, you can reply to this email and have your
r
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/773
-1, this breaks ``TSHttpTxnServerAddrSet``.
---
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 user jaaju opened a pull request:
https://github.com/apache/trafficserver/pull/774
TS-4608: Fix memory leak in ProxyAllocator.cc.
This removes the pre-processor flag TS_USE_FREELIST which is not defined
since commit e56d1f9eb6f55b0257d1fb758b773dea468159ee to make freelists a
GitHub user jacksontj opened a pull request:
https://github.com/apache/trafficserver/pull/773
TS-4622 potential fix
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jacksontj/trafficserver TS-4622
Alternatively you can review and
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69167535
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful genera
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163153
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful genera
Github user zwoop commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163780
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful genera
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163526
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163433
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163229
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69163031
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69162592
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69162190
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69160469
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69160033
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69159927
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69158965
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69158585
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69158201
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69157502
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69157359
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/747#discussion_r69157179
--- Diff: plugins/experimental/remap_purge/remap_purge.c ---
@@ -0,0 +1,335 @@
+/** @file
+
+ Per-remap purge RESTful API for stateful gener
Github user kawaiirice commented on the issue:
https://github.com/apache/trafficserver/pull/769
@SolidWallOfCode
---
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 wish
GitHub user oknet opened a pull request:
https://github.com/apache/trafficserver/pull/772
TS-3726: Expose build_error_response
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/oknet/trafficserver TS-3726
Alternatively you can rev
GitHub user oknet opened a pull request:
https://github.com/apache/trafficserver/pull/771
Proposal: InactivityCop Optimize
Optimize InactivityCop on a heavy load and high level concurrent connection
system.
It is set default inactivity timeout before put vc into open_list.
75 matches
Mail list logo
