[GitHub] trafficserver issue #1398: Configure option to disable CURL
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1398: Configure option to disable CURL
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1392: TS-2095: TS_HAS_LIBZ and TS_HAS_LZMA are always f...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1392: TS-2095: TS_HAS_LIBZ and TS_HAS_LZMA are always f...
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
Build failed in Jenkins: osx-master » clang,osx,release #1369
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 -- [...truncated 1.11 MB...] *** TEST 62 *** STARTING *** *** TEST 62 *** PASSED *** *** TEST 63 *** STARTING *** *** TEST 63 *** PASSED *** *** TEST 64 *** STARTING *** *** TEST 64 *** PASSED *** *** TEST 65 *** STARTING *** *** TEST 65 *** PASSED *** *** TEST 66 *** STARTING *** *** TEST 66 *** PASSED *** *** TEST 67 *** STARTING *** *** TEST 67 *** PASSED *** *** TEST 68 *** STARTING *** *** TEST 68 *** PASSED *** *** TEST 69 *** STARTING *** *** TEST 69 *** PASSED *** *** TEST 70 *** STARTING *** *** TEST 70 *** PASSED *** *** TEST 71 *** STARTING *** *** TEST 71 *** PASSED *** *** TEST 72 *** STARTING *** *** TEST 72 *** PASSED *** *** TEST 73 *** STARTING *** *** TEST 73 *** PASSED *** *** TEST 74 *** STARTING *** *** TEST 74 *** PASSED *** *** TEST 75 *** STARTING *** *** TEST 75 *** PASSED *** *** TEST 76 *** STARTING *** *** TEST 76 *** PASSED *** *** TEST 77 *** STARTING *** *** TEST 77 *** PASSED *** *** TEST 78 *** STARTING *** *** TEST 78 *** PASSED *** *** TEST 79 *** STARTING *** *** TEST 79 *** PASSED *** *** TEST 80 *** STARTING *** *** TEST 80 *** PASSED *** *** TEST 81 *** STARTING *** *** TEST 81 *** PASSED *** *** TEST 82 *** STARTING *** *** TEST 82 *** PASSED *** *** TEST 83 *** STARTING *** *** TEST 83 *** PASSED *** *** TEST 84 *** STARTING *** *** TEST 84 *** PASSED *** *** TEST 85 *** STARTING *** *** TEST 85 *** PASSED *** *** TEST 86 *** STARTING *** *** TEST 86 *** PASSED *** *** TEST 87 *** STARTING *** *** TEST 87 *** PASSED *** *** TEST 88 *** STARTING *** *** TEST 88 *** PASSED *** *** TEST 89 *** STARTING *** *** TEST 89 *** PASSED *** *** TEST 90 *** STARTING *** *** TEST 90 *** PASSED *** *** TEST 91 *** STARTING *** *** TEST 91 *** PASSED *** *** TEST 92 *** STARTING *** *** TEST 92 *** PASSED *** *** TEST 93 *** STARTING *** *** TEST 93 *** PASSED *** *** TEST 94 *** STARTING *** *** TEST 94 *** PASSED *** *** TEST 95 *** STARTING *** *** TEST 95 *** PASSED *** *** TEST 96 *** STARTING *** *** TEST 96 *** PASSED *** *** TEST 97 *** STARTING *** *** TEST 97 *** PASSED *** *** TEST 98 *** STARTING *** *** TEST 98 *** PASSED *** *** TEST 99 *** STARTING *** *** TEST 99 *** PASSED *** *** TEST 100 *** STARTING *** *** TEST 100 *** PASSED *** *** TEST 101 *** STARTING *** *** TEST 101 *** PASSED *** *** TEST 102 *** STARTING *** *** TEST 102 *** PASSED *** *** TEST 103 *** STARTING *** *** TEST 103 *** PASSED *** *** TEST 104 *** STARTING *** *** TEST 104 *** PASSED *** *** TEST 105 *** STARTING *** *** TEST 105 *** PASSED *** *** TEST 106 *** STARTING *** *** TEST 106 *** PASSED *** *** TEST 107 *** STARTING *** *** TEST 107 *** PASSED *** *** TEST 108 *** STARTING *** *** TEST 108 *** PASSED *** *** TEST 109 *** STARTING *** *** TEST 109 *** PASSED *** *** TEST 110 *** STARTING *** *** TEST 110 *** PASSED *** *** TEST 111 *** STARTING *** *** TEST 111 *** PASSED *** *** TEST 112 *** STARTING *** *** TEST 112 *** PASSED *** *** TEST 113 *** STARTING *** *** TEST 113 *** PASSED *** *** TEST 114 *** STARTING *** *** TEST 114 *** PASSED *** *** TEST 115 *** STARTING *** *** TEST 115 *** PASSED *** *** TEST 116 *** STARTING *** *** TEST 116 *** PASSED *** *** TEST 117 *** STARTING *** *** TEST 117 *** PASSED *** *** TEST 118 *** STARTING *** *** TEST 118 *** PASSED *** *** TEST 119REGRESSION_RESULT PARENTSELECTION: FAILED REGRESSION_TEST DONE: FAILED *** STARTING *** *** TEST 119 *** PASSED *** *** TEST 120 *** STARTING *** *** TEST 120 *** PASSED *** *** TEST 121 *** STARTING *** *** TEST 121 *** PASSED *** *** TEST 122 *** STARTING *** *** TEST 122 *** PASSED *** *** TEST 123 *** STARTING *** *** TEST 123 *** PASSED *** *** TEST 124 *** STARTING *** *** TEST 124 *** PASSED *** *** TEST 125 *** STARTING *** *** TEST 125 *** PASSED *** *** TEST 126 *** STARTING *** *** TEST 126 *** PASSED *** *** TEST 127 *** STARTING *** *** TEST 127 *** PASSED *** *** TEST 128 *** STARTING *** *** TEST 128 *** PASSED *** *** TEST 129 *** STARTING *** *** TEST 129 *** PASSED *** *** TEST 130 *** STARTING *** *** TEST 130 *** PASSED *** *** TEST 131 *** STARTING *** *** TEST 131 *** PASSED *** *** TEST 132 *** STARTING *** *** TEST 132 *** PASSED *** *** TEST 133 *** STARTING *** *** TEST 133 *** PASSED *** *** TEST 134 *** STARTING *** *** TEST 134 *** PASSED *** *** TEST 135 *** STARTING *** *** TEST 135 *** PASSED *** *** TEST 136 *** STARTING *** *** TEST 136 *** PASSED *** *** TEST 137 *** STARTING *** *** TEST 137 *** PASSED *** *** TEST 138 *** STARTING *** *** TEST 138 *** PASSED *** *** TEST 139 *** STARTING *** *** TEST 139 *** PASSED *** *** TEST 140 *** STARTING *** *** TEST 140 *** PASSED *** *** TEST 141 *** STARTING *** *** TEST 141 ***
[GitHub] trafficserver issue #1398: Configure option to disable CURL
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1391: Allow configurable body factory response m...
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1398: Configure option to disable CURL
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1398: Configure option to disable CURL
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 can review and apply these changes as the patch at: https://github.com/apache/trafficserver/pull/1398.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1398 commit 9865edc02b86a549c7d5a1b8f66426bd1fbb2d50 Author: Bryan Call Date: 2017-01-31T00:23:02Z Configure option to disable CURL --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1397: TS-1768: AC_SEARCH_LIBS vs. AC_CHECK_LIB
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 libraries? Is it memory consumption? --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1392: TS-2095: TS_HAS_LIBZ and TS_HAS_LZMA are always f...
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", BUILD_GROUP, json); print_feature("BUILD_NUMBER", BUILD_NUMBER, json); - print_feature("TS_HAS_LIBZ", TS_HAS_LIBZ, json); - print_feature("TS_HAS_LZMA", TS_HAS_LZMA, json); +#ifdef HAVE_ZLIB_H + print_feature("HAVE_ZLIB_H", HAVE_ZLIB_H, json); +#endif +#ifdef HAVE_LZMA_H + print_feature("HAVE_LZMA_H", HAVE_LZMA_H, json); +#endif print_feature("TS_HAS_JEMALLOC", TS_HAS_JEMALLOC, json); print_feature("TS_HAS_TCMALLOC", TS_HAS_TCMALLOC, json); print_feature("TS_HAS_IN6_IS_ADDR_UNSPECIFIED", TS_HAS_IN6_IS_ADDR_UNSPECIFIE D, json); ``` Fixing zlibh and lzmah would be very easy! I'd just sooner get rid of them ... What irritates me is that they reimplement something that already works automatically and more standardly: HAVE_ZLIB_H and HAVE_LZMA_H. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1392: TS-2095: TS_HAS_LIBZ and TS_HAS_LZMA are always f...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1392: TS-2095: TS_HAS_LIBZ and TS_HAS_LZMA are always f...
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1397: TS-1768: AC_SEARCH_LIBS vs. AC_CHECK_LIB
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1397: TS-1768: AC_SEARCH_LIBS vs. AC_CHECK_LIB
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1397: TS-1768: AC_SEARCH_LIBS vs. AC_CHECK_LIB
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 does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1397: TS-1768: AC_SEARCH_LIBS vs. AC_CHECK_LIB
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 library isn't required. So prefer AC_SEARCH_LIBS. Use AC_CHECK_LIB only if 1) we want to leave LIBS alone, and 2) the function isn't likely to be provided by some other library -- the standard C library, for example. If there's a chance that the function might already be available, use AC_SEARCH_LIBS, but save and restore LIBS as needed. You can merge this pull request into a Git repository by running: $ git pull https://github.com/jablko/trafficserver libs Alternatively you can review and apply these changes as the patch at: https://github.com/apache/trafficserver/pull/1397.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1397 commit cc4b8c72ed5ad7059ad5c6f44ba2616556f69172 Author: Jack Bates Date: 2017-01-30T22:28:28Z 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 library isn't required. So prefer AC_SEARCH_LIBS. Use AC_CHECK_LIB only if 1) we want to leave LIBS alone, and 2) the function isn't likely to be provided by some other library -- the standard C library, for example. If there's a chance that the function might already be available, use AC_SEARCH_LIBS, but save and restore LIBS as needed. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1396: check if any of client cert path or file i...
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 is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1396: check if any of client cert path or file is null
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1396: check if any of client cert path or file is null
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1396: check if any of client cert path or file is null
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, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1396: check if any of client cert path or file is null
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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1396: check if any of client cert path or file i...
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 https://github.com/persiaAziz/trafficserver master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/trafficserver/pull/1396.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #1396 commit 625485b24a8a343fd089e5f063910a78e051c4db Author: Persia Aziz Date: 2017-01-30T22:35:52Z check if any of client cert path or file is null --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1393: Trivial Autoconf cleanup
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1393: Trivial Autoconf cleanup
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1332: Updates to cache documentation and Cache Tool.
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1332: Updates to cache documentation and Cache Tool.
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...
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 relative paths, otherwise the current working directory of whatever does the loading becomes an ABI. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1248: TS-5070 Add configuration variables to set file p...
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 as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1248: TS-5070 Add configuration variables to set file p...
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 feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1393: Trivial Autoconf cleanup
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 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
Jenkins build is back to normal : debian_8-master » gcc,debian_8,debug #1284
https://ci.trafficserver.apache.org/job/debian_8-master/compiler=gcc,label=debian_8,type=debug/1284/display/redirect
Jenkins build is back to normal : debian_7-master » gcc,debian_7,debug #2415
https://ci.trafficserver.apache.org/job/debian_7-master/compiler=gcc,label=debian_7,type=debug/2415/display/redirect
[GitHub] trafficserver issue #1392: TS-2095: TS_HAS_LIBZ and TS_HAS_LZMA are always f...
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 project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1365: TS-4896: TSHttp***ClientAddrGet/TSHttp***Incoming...
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 as far as I can tell, this is to save way the IP addresses since the VC goes away. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
Jenkins build is back to normal : osx-master » clang,osx,release #1366
https://ci.trafficserver.apache.org/job/osx-master/compiler=clang,label=osx,type=release/1366/
[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1321: 6.2.x TS-4435 and TS-5059: Fixes for building wit...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1321: 6.2.x TS-4435 and TS-5059: Fixes for building wit...
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...
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 renamed to proxy.config.ssl.client.cert.file. * proxy.config.ssl.client.cert.file is overridable * The user needs to specify the fullpath in proxy.config.ssl.client.cert.file. * It does not check if the given path is absolute or not because if the path is not absolute then concatenating (default)path and filename is again going to end up in allocating memory on each remap. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1357: TS-5022: reduce memory allocation in clientcert l...
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1386: ATS hangs, possibly because of OpenSSL 1.1.0
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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1389: SSL_CTX_remove_session() and external cach...
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1393: Trivial Autoconf cleanup
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: ``` AC_SEARCH_LIBS([lzma_code], [lzma], [lzma_have_libs=1]) if test "$lzma_have_libs" != "0"; then AC_CHECK_HEADERS(lzma.h, [lzma_have_headers=1]) fi if test "$lzma_have_headers" != "0"; then AC_SUBST(LIBLZMA, [-llzma]) else enable_lzma=no CPPFLAGS=$saved_cppflags LDFLAGS=$saved_ldflags fi ``` --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1391: Allow configurable body factory response max size
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 so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #210: TS-3607: Integrate ats_pagespeed into the automake...
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 your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #210: TS-3607: Integrate ats_pagespeed into the a...
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #761: TS-4332: proxy.config.net.connections_throt...
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #761: TS-4332: proxy.config.net.connections_throttle sho...
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 well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1395: Update client streams count in Http2ConnectionSta...
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 does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1387: Add diags log message when cache wraps.
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 which might obviate the utility of this is making the wrap stat persistent, which it is not currently. --- 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 enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1248: TS-5070 Add configuration variables to set...
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver pull request #1365: TS-4896: TSHttp***ClientAddrGet/TSHttp***I...
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 feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1332: Updates to cache documentation and Cache Tool.
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 appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] trafficserver issue #1332: Updates to cache documentation and Cache Tool.
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 reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---