Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1423/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1316/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SolidWallOfCode commented on the issue:
https://github.com/apache/trafficserver/pull/1387
I put this in to Y!'s fork but I have mixed feelings about it. It has been
handy for our operations team but I'm not sure it is of sufficient general
utility. One outstanding request
Github user shinrich closed the pull request at:
https://github.com/apache/trafficserver/pull/1365
---
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
Github user shinrich closed the pull request at:
https://github.com/apache/trafficserver/pull/1248
---
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
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1391
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1317/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jablko closed the issue at:
https://github.com/apache/trafficserver/issues/1386
---
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 is
Github user jablko closed the pull request at:
https://github.com/apache/trafficserver/pull/1389
---
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
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/1391
[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 feature
enabled and wishes
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/761
Please reopen or create a new pull request if you still want to resolve
this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user bryancall closed the pull request at:
https://github.com/apache/trafficserver/pull/761
---
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
Github user bryancall closed the pull request at:
https://github.com/apache/trafficserver/pull/210
---
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
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/210
Please reopen or create another pull request if you want to resolve this.
---
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 user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/1391
[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 feature
enabled and wishes
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1395
Giving to @PSUdaemon, this is for him to consider for backport.
---
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 user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1391
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1424/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1391
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1318/ 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/1393
Nice catch on AC_SEARCH_LIBS! Question, i wonder if we do this wrong in
other places? Are we adding things to LIBS when we shouldn't?
There are several places where we do things like:
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/1391
Please run clang-format on the code. It is not passing 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
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/1248
Please squash commits before merging.
---
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 zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1248
For future references, this really should have been squashed down to one
commit before merging.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user persiaAziz opened a pull request:
https://github.com/apache/trafficserver/pull/1396
check if any of client cert path or file is null
fix for client cert loading problem
You can merge this pull request into a Git repository by running:
$ git pull
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1396
[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 feature
enabled and wishes so,
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1396
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1430/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1396
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1323/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user jpeach commented on the issue:
https://github.com/apache/trafficserver/pull/1357
Whilst the configuration of SSL keys and certificates is a bit unfortunate,
I think that it is important to be consistent.
Separately, you have to define the base directory for
Github user persiaAziz commented on the issue:
https://github.com/apache/trafficserver/pull/1396
please review @zwoop @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
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1357
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1427/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1357
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1321/ 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/1357
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1428/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1393
Thanks for reviewing this @zwoop! I'm going to land it and see about fixing
the issue you spotted in a separate PR.
---
If your project is set up for it, you can reply to this email and have
Github user jablko closed the pull request at:
https://github.com/apache/trafficserver/pull/1393
---
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
https://ci.trafficserver.apache.org/job/debian_8-master/compiler=gcc,label=debian_8,type=debug/1284/display/redirect
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1393
In fact, taking a quick look, tools like traffic_top is getting all the
"optional" libraries, even though they are highly unlikely to use 'em? Like,
libhwloc.
---
If your project is set up
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
FreeBSD build *failed*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1429/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1332
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1322/ 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/1357
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1320/ for details.
---
If your project is set up for it, you can reply to this email and have your
reply
https://ci.trafficserver.apache.org/job/osx-master/compiler=clang,label=osx,type=release/1366/
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1365
Nit picky, and too late, but I kinda wish these two new variables would
have been named "saved_XXX" instead of "cached_XXX". Cached make it sounds like
it's an optimization of some sort, but
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1321
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1319/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1391
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1425/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user persiaAziz commented on the issue:
https://github.com/apache/trafficserver/pull/1357
@SolidWallOfCode, @shinrich Please review.
In summary,
* proxy.config.ssl.client.cert.path has been completely removed.
* proxy.config.ssl.client.cert.filename has been
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1321
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1426/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1392
I suspect we might want to backport a fix for this to 7.1.x as well.
---
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
https://ci.trafficserver.apache.org/job/debian_7-master/compiler=gcc,label=debian_7,type=debug/2415/display/redirect
Github user zwoop commented on the issue:
https://github.com/apache/trafficserver/pull/1397
Wonder if we should bother back porting these to 7.1.x ? Wdyt ?
---
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 user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1397
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1324/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1391
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1326/ 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/1392
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1435/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1398
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1436/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1397
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1431/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1325/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Would this latest change work? How does traffic_layout --features get used?
```Diff
print_feature("BUILD_PERSON", BUILD_PERSON, json);
print_feature("BUILD_GROUP",
GitHub user jablko opened a pull request:
https://github.com/apache/trafficserver/pull/1397
TS-1768: AC_SEARCH_LIBS vs. AC_CHECK_LIB
AC_SEARCH_LIBS always adds -llibrary to LIBS, but AC_CHECK_LIB doesn't
check that the function isn't already available -- in which case the
Github user zwoop closed the pull request at:
https://github.com/apache/trafficserver/pull/1396
---
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 bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/1391
[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 feature
enabled and wishes
Github user jablko commented on the issue:
https://github.com/apache/trafficserver/pull/1397
> Wonder if we should bother back porting these to 7.1.x ? Wdyt ?
It's probably safe. How serious is the problem it's fixing? What's the
downside of linking with unnecessary
GitHub user bryancall opened a pull request:
https://github.com/apache/trafficserver/pull/1398
Configure option to disable CURL
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bryancall/trafficserver no_curl
Alternatively you
Github user bryancall commented on the issue:
https://github.com/apache/trafficserver/pull/1391
Still needs a clang-format. I will merge it and clang-format it once it is
committed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1398
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1327/ 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/1398
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1434/ for details.
---
If your project is set up for it, you can reply to this email and have
https://ci.trafficserver.apache.org/job/osx-master/compiler=clang,label=osx,type=release/1369/display/redirectChanges:
[Leif Hedstrom] check if any of client cert path or file is null
[Bryan Call] Allow configurable body factory response max size
[Bryan Call] clang-format
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1392
Linux build *successful*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1328/ for details.
---
If your project is set up for it, you can reply to this email and have your
Github user atsci commented on the issue:
https://github.com/apache/trafficserver/pull/1398
Linux build *failed*! See
https://ci.trafficserver.apache.org/job/Github-Linux/1329/ 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/1392
FreeBSD build *successful*! See
https://ci.trafficserver.apache.org/job/Github-FreeBSD/1432/ for details.
---
If your project is set up for it, you can reply to this email and have
Github user bryancall closed the pull request at:
https://github.com/apache/trafficserver/pull/1391
---
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
66 matches
Mail list logo
