You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ma...@apache.org on 2018/08/01 01:01:48 UTC

[trafficserver] branch quic-latest updated (45a3512 -> 38104e9)

This is an automated email from the ASF dual-hosted git repository.

masaori pushed a change to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git.


    from 45a3512      Add be32toh and htobe32
     new 6365480  Fix unit test of QUICPakcet
     add ba93177  Enhances detection of luajit
     add 4d4a2d2   Removes compiler flag overrides related to obsolete intree LuaJIT build
     add 113fb90  Cleans up LDADDs in traffic_server
     add c3f0582  Cleans up brotli compiler and linker flags
     add 84f429f  Doc: Tweak to fix doc issues in IntrusiveDList documentation.
     add 77fb66b  Add generic "guard" class (PostScript) for exception and early function return safety.
     add fbb01fe  Add information about the connect failure and source port
     add 441f5f6  Fixed broken sphinx version check on MacOS
     add e632e46  Fixes spelling
     add a9157d5  Test: Convert test_Regex.cc to Catch.
     add e57679c  Fixed http/2 issue with Http2Stream being updated after it was destroyed
     add ef785f2  Clearout shutdown_cont_event when the event has been processed
     add 31e4d28  Testing: Convert test_History.cc to Catch.
     add 22e273d  TextView: More unit tests about separators and tokens.
     add e4a94b9  Remove Http2ConnectionState::continued_buffer
     add 4c34cbc  Make negative caching accept configured error status codes
     add 6c8cd24  Check version in the snap file on ATS upgrade
     add a80f072  Minor update on README's directory structure
     add 7acfcfb  Fixes some var-args missing va_start/end
     add a8ec551  Do not do DNS to origin if the object is HIT-STALE and parent exists
     add 13a26ac  Bug fix in microServer threading and reorganize code
     add 2088331  Fix issue caused by snap file
     add 37aa9fe  Runroot: Add feature to use runroot by providing yaml file
     add 74f03c7  Adds new logging tags: nhi and nhp
     add 175143e  TS-4765: Removes previously deprecated cqbl and pqbl log tags
     add 67a6e05  TextView: Unit tests for token handling on single characters, and with trim.
     add f78dfc4  Test: Convert test_Ptr.cc to Catch.
     add 65cb226  Removes old commented-out code
     add cf353b4  Revert "Fix crashes from early post return fix."
     add 33543d8  Revert "Fix post_error test by initializating the address length of the accept call."
     add 769ca45  Revert "Fix post process to propagate early server response."
     add 7b69a90  Dockerfile for CentOS/Fedora, i.e. yum dependencies
     add 52738fb  Revert "Do not do DNS to origin if the object is HIT-STALE and parent exists"
     add 92accb1  fix another crash on shutdown and add unlikely
     add f38afaa  Modifies init script to add start/reload hooks
     add df6c2fb  For PostScript class, remove problematic parameter forwarding to functor (rely on lambda capturing instead).
     add 8aec34a  Add support for 'fwd' value to X-Debug header, and move to later hook any deletion of X-Debug header from client request.
     add bd7741f  Fixes ticket loading from filesystems without a mtime
     add 044f9d6  Removes more references to traffic_cop and cop related functionality
     add 78d7c3f  Fixing copy paste error in SNI yaml parsing
     add cee6d61  Change the defauilt connect_ports docs to reflect the code
     add 5c23970  Removes remnants of dprintf support
     add c6cf6aa  Fixed Spelling.
     add 2ded013  Updates the default ciphers to avoid weak ciphers(non FS)
     new 38104e9  Merge branch 'asf/master' into quic-latest

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 CMakeLists.txt                                     |   7 +-
 README                                             |  13 +-
 build/brotli.m4                                    |  18 +--
 build/luajit.m4                                    | 176 +++++++++++++++++++++
 ci/docker/yum/Dockerfile                           |  68 ++++++++
 configs/records.config.default.in                  |   2 +-
 configure.ac                                       | 115 ++++----------
 doc/.tx/config                                     |   5 -
 doc/admin-guide/files/records.config.en.rst        |  11 +-
 doc/admin-guide/logging/formatting.en.rst          |   4 +
 .../monitoring/statistics/core/general.en.rst      |   3 -
 doc/admin-guide/performance/index.en.rst           |   4 +-
 doc/admin-guide/plugins/lua.en.rst                 |   6 +-
 doc/admin-guide/plugins/xdebug.en.rst              |   7 +-
 doc/appendices/command-line/traffic_server.en.rst  |   2 -
 doc/checkvers.sh                                   |  40 -----
 .../internal-libraries/intrusive-list.en.rst       |  65 +++++---
 iocore/net/SSLConfig.cc                            |   7 +-
 iocore/net/UnixNetAccept.cc                        |   2 +-
 iocore/net/UnixNetProcessor.cc                     |   3 +
 iocore/net/YamlSNIConfig.cc                        |   2 +-
 iocore/net/quic/test/test_QUICPacket.cc            |  13 +-
 lib/perl/lib/Apache/TS/AdminClient.pm              |   1 +
 lib/records/Makefile.am                            |   7 +-
 lib/records/P_RecCore.cc                           |  25 +++
 lib/records/P_RecFile.h                            |   3 +
 lib/records/P_RecMessage.h                         |   1 -
 lib/records/RecCore.cc                             |   2 +-
 lib/records/RecFile.cc                             |  36 +++++
 lib/records/RecLocal.cc                            |   9 --
 lib/records/RecMessage.cc                          |   4 +-
 lib/records/RecProcess.cc                          |  19 ---
 lib/ts/History.h                                   |   1 +
 lib/ts/Makefile.am                                 |  10 +-
 lib/ts/PostScript.h                                |  65 ++++++++
 lib/ts/TextView.h                                  |  28 ++++
 lib/ts/ink_error.h                                 |   2 -
 lib/ts/ink_res_init.cc                             |   2 +-
 lib/ts/ink_sock.cc                                 |   1 -
 lib/ts/ink_sprintf.cc                              |   1 +
 lib/ts/runroot.cc                                  |  81 +++++-----
 lib/ts/runroot.h                                   |   4 -
 lib/ts/test_History.cc                             | 111 -------------
 lib/ts/test_Ptr.cc                                 |  99 ------------
 lib/ts/unit-tests/test_History.cc                  | 117 ++++++++++++++
 lib/ts/unit-tests/test_IntrusiveDList.cc           |  27 +++-
 ...{test_scoped_resource.cc => test_PostScript.cc} |  69 +++++---
 lib/ts/unit-tests/test_Ptr.cc                      |  80 ++++++++++
 lib/ts/{ => unit-tests}/test_Regex.cc              |  35 ++--
 lib/ts/unit-tests/test_TextView.cc                 |  63 +++++++-
 mgmt/Cop.h                                         |  27 ----
 mgmt/RecordsConfig.cc                              |   4 +-
 mgmt/api/include/mgmtapi.h                         |   2 +-
 plugins/compress/Makefile.inc                      |   4 +-
 plugins/lua/Makefile.inc                           |   2 +-
 plugins/xdebug/xdebug.cc                           | 148 ++++++++++++++---
 proxy/hdrs/MIME.h                                  |   4 +-
 proxy/http/HttpBodyFactory.cc                      |   1 -
 proxy/http/HttpBodyFactory.h                       |  10 +-
 proxy/http/HttpConfig.cc                           |  54 +++++++
 proxy/http/HttpConfig.h                            |   6 +
 proxy/http/HttpSM.cc                               |  31 +---
 proxy/http/HttpTransact.cc                         |  32 ++--
 proxy/http/HttpTransact.h                          |  11 --
 proxy/http2/Http2ConnectionState.cc                |   2 +
 proxy/http2/Http2ConnectionState.h                 |  10 +-
 proxy/http2/Http2Stream.cc                         |   4 +-
 proxy/logging/Log.cc                               |  20 +--
 proxy/logging/LogAccess.cc                         |  13 ++
 proxy/logging/LogAccess.h                          |   2 +
 proxy/logging/LogAccessHttp.cc                     |  16 ++
 proxy/logging/LogAccessHttp.h                      |   2 +
 rc/trafficserver.in                                |  35 ++++
 src/traffic_ctl/config.cc                          |   5 -
 src/traffic_layout/engine.cc                       |  39 ++++-
 src/traffic_manager/traffic_manager.cc             |   1 -
 src/traffic_server/CoreUtils.cc                    |   4 -
 src/traffic_server/HostStatus.cc                   |   2 +-
 src/traffic_server/Makefile.inc                    |  11 +-
 src/traffic_server/SocksProxy.cc                   |   5 -
 src/traffic_server/traffic_server.cc               |  23 +--
 src/traffic_top/traffic_top.cc                     |   2 -
 tests/gold_tests/autest-site/microserver.test.ext  |   7 +-
 .../pluginTest/xdebug/x_remap/{two.in => fwd1.in}  |   2 +-
 .../pluginTest/xdebug/x_remap/{two.in => fwd2.in}  |   2 +-
 .../pluginTest/xdebug/x_remap/{two.in => fwd3.in}  |   2 +-
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd4.in |   4 +
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd5.in |   4 +
 .../gold_tests/pluginTest/xdebug/x_remap/out.gold  | 139 +++++++++++-----
 .../xdebug/x_remap/x_remap-observer.py}            |  16 +-
 .../pluginTest/xdebug/x_remap/x_remap.gold         |  16 ++
 .../pluginTest/xdebug/x_remap/x_remap.test.py      |  20 ++-
 tests/gold_tests/post_error/create_post_body.py    |  24 ---
 tests/gold_tests/post_error/gold/post_error.gold   |   3 -
 tests/gold_tests/post_error/post_error.test.py     |  69 --------
 tests/gold_tests/post_error/post_server.c          | 105 ------------
 tests/gold_tests/runroot/runroot_error.test.py     |   4 +-
 tests/tools/microServer/uWServer.py                |  25 ++-
 98 files changed, 1460 insertions(+), 990 deletions(-)
 create mode 100644 build/luajit.m4
 create mode 100644 ci/docker/yum/Dockerfile
 delete mode 100755 doc/checkvers.sh
 create mode 100644 lib/ts/PostScript.h
 delete mode 100644 lib/ts/test_History.cc
 delete mode 100644 lib/ts/test_Ptr.cc
 create mode 100644 lib/ts/unit-tests/test_History.cc
 copy lib/ts/unit-tests/{test_scoped_resource.cc => test_PostScript.cc} (52%)
 create mode 100644 lib/ts/unit-tests/test_Ptr.cc
 rename lib/ts/{ => unit-tests}/test_Regex.cc (53%)
 delete mode 100644 mgmt/Cop.h
 copy tests/gold_tests/pluginTest/xdebug/x_remap/{two.in => fwd1.in} (57%)
 copy tests/gold_tests/pluginTest/xdebug/x_remap/{two.in => fwd2.in} (55%)
 copy tests/gold_tests/pluginTest/xdebug/x_remap/{two.in => fwd3.in} (54%)
 create mode 100644 tests/gold_tests/pluginTest/xdebug/x_remap/fwd4.in
 create mode 100644 tests/gold_tests/pluginTest/xdebug/x_remap/fwd5.in
 copy tests/gold_tests/{headers/normalize_ae_observer.py => pluginTest/xdebug/x_remap/x_remap-observer.py} (73%)
 create mode 100644 tests/gold_tests/pluginTest/xdebug/x_remap/x_remap.gold
 delete mode 100644 tests/gold_tests/post_error/create_post_body.py
 delete mode 100644 tests/gold_tests/post_error/gold/post_error.gold
 delete mode 100644 tests/gold_tests/post_error/post_error.test.py
 delete mode 100644 tests/gold_tests/post_error/post_server.c


[trafficserver] 01/02: Fix unit test of QUICPakcet

Posted by ma...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit 636548084557512da0a7bff9e0f59d85ece9a9a7
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Wed Aug 1 09:51:57 2018 +0900

    Fix unit test of QUICPakcet
---
 iocore/net/quic/test/test_QUICPacket.cc | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/iocore/net/quic/test/test_QUICPacket.cc b/iocore/net/quic/test/test_QUICPacket.cc
index e1be5de..a8bcc4b 100644
--- a/iocore/net/quic/test/test_QUICPacket.cc
+++ b/iocore/net/quic/test/test_QUICPacket.cc
@@ -187,7 +187,9 @@ TEST_CASE("Encoded Packet Number Length", "[quic]")
 {
   QUICPacketNumber base = 0x6afa2f;
 
-  CHECK(QUICPacket::calc_packet_number_len(0x6b4264, base) == 2);
+  // To be clarify: Is "As a result..." sentence missing "twice" ?
+  //
+  // CHECK(QUICPacket::calc_packet_number_len(0x6b2d79, base) == 2);
   CHECK(QUICPacket::calc_packet_number_len(0x6bc107, base) == 4);
 }
 
@@ -203,12 +205,12 @@ TEST_CASE("Encoding Packet Number", "[quic]")
 TEST_CASE("Decoding Packet Number 1", "[quic]")
 {
   QUICPacketNumber dst  = 0;
-  QUICPacketNumber src  = 0x1f94;
+  QUICPacketNumber src  = 0x9b3;
   size_t len            = 2;
   QUICPacketNumber base = 0xaa82f30e;
 
   QUICPacket::decode_packet_number(dst, src, len, base);
-  CHECK(dst == 0xaa831f94);
+  CHECK(dst == 0xaa8309b3);
 }
 
 TEST_CASE("Decoding Packet Number 2", "[quic]")
@@ -219,7 +221,7 @@ TEST_CASE("Decoding Packet Number 2", "[quic]")
   QUICPacketNumber base = 0x18bf54f0;
 
   QUICPacket::decode_packet_number(dst, src, len, base);
-  CHECK(dst == 0x18bf54f1);
+  CHECK(dst == 0x18bf5571);
 }
 
 TEST_CASE("Decoding Packet Number 3", "[quic]")
@@ -228,6 +230,7 @@ TEST_CASE("Decoding Packet Number 3", "[quic]")
   QUICPacketNumber src  = 0x5694;
   size_t len            = 2;
   QUICPacketNumber base = 0x44D35695;
+
   QUICPacket::decode_packet_number(dst, src, len, base);
-  CHECK(dst == 0x44D35694);
+  CHECK(dst == 0x44d39694);
 }


[trafficserver] 02/02: Merge branch 'asf/master' into quic-latest

Posted by ma...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git

commit 38104e981d592bc95271f8fb3eca95cb29b16443
Merge: 6365480 2ded013
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Wed Aug 1 09:58:19 2018 +0900

    Merge branch 'asf/master' into quic-latest
    
    * asf/master: (44 commits)
      Updates the default ciphers to avoid weak ciphers(non FS)
      Fixed Spelling.
      Removes remnants of dprintf support
      Change the defauilt connect_ports docs to reflect the code
      Fixing copy paste error in SNI yaml parsing
      Removes more references to traffic_cop and cop related functionality
      Fixes ticket loading from filesystems without a mtime
      Add support for 'fwd' value to X-Debug header, and move to later hook any deletion of X-Debug header from client request.
      For PostScript class, remove problematic parameter forwarding to functor (rely on lambda capturing instead).
      Modifies init script to add start/reload hooks
      fix another crash on shutdown and add unlikely
      Revert "Do not do DNS to origin if the object is HIT-STALE and parent exists"
      Dockerfile for CentOS/Fedora, i.e. yum dependencies
      Revert "Fix post process to propagate early server response."
      Revert "Fix post_error test by initializating the address length of the accept call."
      Revert "Fix crashes from early post return fix."
      Removes old commented-out code
      Test: Convert test_Ptr.cc to Catch.
      TextView: Unit tests for token handling on single characters, and with trim.
      TS-4765: Removes previously deprecated cqbl and pqbl log tags
      ...

 CMakeLists.txt                                     |   7 +-
 README                                             |  13 +-
 build/brotli.m4                                    |  18 +--
 build/luajit.m4                                    | 176 +++++++++++++++++++++
 ci/docker/yum/Dockerfile                           |  68 ++++++++
 configs/records.config.default.in                  |   2 +-
 configure.ac                                       | 115 ++++----------
 doc/.tx/config                                     |   5 -
 doc/admin-guide/files/records.config.en.rst        |  11 +-
 doc/admin-guide/logging/formatting.en.rst          |   4 +
 .../monitoring/statistics/core/general.en.rst      |   3 -
 doc/admin-guide/performance/index.en.rst           |   4 +-
 doc/admin-guide/plugins/lua.en.rst                 |   6 +-
 doc/admin-guide/plugins/xdebug.en.rst              |   7 +-
 doc/appendices/command-line/traffic_server.en.rst  |   2 -
 doc/checkvers.sh                                   |  40 -----
 .../internal-libraries/intrusive-list.en.rst       |  65 +++++---
 iocore/net/SSLConfig.cc                            |   7 +-
 iocore/net/UnixNetAccept.cc                        |   2 +-
 iocore/net/UnixNetProcessor.cc                     |   3 +
 iocore/net/YamlSNIConfig.cc                        |   2 +-
 lib/perl/lib/Apache/TS/AdminClient.pm              |   1 +
 lib/records/Makefile.am                            |   7 +-
 lib/records/P_RecCore.cc                           |  25 +++
 lib/records/P_RecFile.h                            |   3 +
 lib/records/P_RecMessage.h                         |   1 -
 lib/records/RecCore.cc                             |   2 +-
 lib/records/RecFile.cc                             |  36 +++++
 lib/records/RecLocal.cc                            |   9 --
 lib/records/RecMessage.cc                          |   4 +-
 lib/records/RecProcess.cc                          |  19 ---
 lib/ts/History.h                                   |   1 +
 lib/ts/Makefile.am                                 |  10 +-
 lib/ts/PostScript.h                                |  65 ++++++++
 lib/ts/TextView.h                                  |  28 ++++
 lib/ts/ink_error.h                                 |   2 -
 lib/ts/ink_res_init.cc                             |   2 +-
 lib/ts/ink_sock.cc                                 |   1 -
 lib/ts/ink_sprintf.cc                              |   1 +
 lib/ts/runroot.cc                                  |  81 +++++-----
 lib/ts/runroot.h                                   |   4 -
 lib/ts/test_History.cc                             | 111 -------------
 lib/ts/test_Ptr.cc                                 |  99 ------------
 lib/ts/unit-tests/test_History.cc                  | 117 ++++++++++++++
 lib/ts/unit-tests/test_IntrusiveDList.cc           |  27 +++-
 mgmt/Cop.h => lib/ts/unit-tests/test_PostScript.cc |  62 +++++++-
 lib/ts/unit-tests/test_Ptr.cc                      |  80 ++++++++++
 lib/ts/{ => unit-tests}/test_Regex.cc              |  35 ++--
 lib/ts/unit-tests/test_TextView.cc                 |  63 +++++++-
 mgmt/RecordsConfig.cc                              |   4 +-
 mgmt/api/include/mgmtapi.h                         |   2 +-
 plugins/compress/Makefile.inc                      |   4 +-
 plugins/lua/Makefile.inc                           |   2 +-
 plugins/xdebug/xdebug.cc                           | 148 ++++++++++++++---
 proxy/hdrs/MIME.h                                  |   4 +-
 proxy/http/HttpBodyFactory.cc                      |   1 -
 proxy/http/HttpBodyFactory.h                       |  10 +-
 proxy/http/HttpConfig.cc                           |  54 +++++++
 proxy/http/HttpConfig.h                            |   6 +
 proxy/http/HttpSM.cc                               |  31 +---
 proxy/http/HttpTransact.cc                         |  32 ++--
 proxy/http/HttpTransact.h                          |  11 --
 proxy/http2/Http2ConnectionState.cc                |   2 +
 proxy/http2/Http2ConnectionState.h                 |  10 +-
 proxy/http2/Http2Stream.cc                         |   4 +-
 proxy/logging/Log.cc                               |  20 +--
 proxy/logging/LogAccess.cc                         |  13 ++
 proxy/logging/LogAccess.h                          |   2 +
 proxy/logging/LogAccessHttp.cc                     |  16 ++
 proxy/logging/LogAccessHttp.h                      |   2 +
 rc/trafficserver.in                                |  35 ++++
 src/traffic_ctl/config.cc                          |   5 -
 src/traffic_layout/engine.cc                       |  39 ++++-
 src/traffic_manager/traffic_manager.cc             |   1 -
 src/traffic_server/CoreUtils.cc                    |   4 -
 src/traffic_server/HostStatus.cc                   |   2 +-
 src/traffic_server/Makefile.inc                    |  11 +-
 src/traffic_server/SocksProxy.cc                   |   5 -
 src/traffic_server/traffic_server.cc               |  23 +--
 src/traffic_top/traffic_top.cc                     |   2 -
 tests/gold_tests/autest-site/microserver.test.ext  |   7 +-
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd1.in |   4 +
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd2.in |   4 +
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd3.in |   4 +
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd4.in |   4 +
 tests/gold_tests/pluginTest/xdebug/x_remap/fwd5.in |   4 +
 .../gold_tests/pluginTest/xdebug/x_remap/out.gold  | 139 +++++++++++-----
 .../xdebug/x_remap/x_remap-observer.py}            |  21 ++-
 .../pluginTest/xdebug/x_remap/x_remap.gold         |  16 ++
 .../pluginTest/xdebug/x_remap/x_remap.test.py      |  20 ++-
 tests/gold_tests/post_error/gold/post_error.gold   |   3 -
 tests/gold_tests/post_error/post_error.test.py     |  69 --------
 tests/gold_tests/post_error/post_server.c          | 105 ------------
 tests/gold_tests/runroot/runroot_error.test.py     |   4 +-
 tests/tools/microServer/uWServer.py                |  25 ++-
 95 files changed, 1479 insertions(+), 911 deletions(-)

diff --cc lib/ts/Makefile.am
index 35d6160,ef44472..2e3f576
--- a/lib/ts/Makefile.am
+++ b/lib/ts/Makefile.am
@@@ -287,14 -275,12 +286,17 @@@ test_tslib_SOURCES = 
  	unit-tests/test_MemSpan.cc \
  	unit-tests/test_MemArena.cc \
  	unit-tests/test_MT_hashtable.cc \
+ 	unit-tests/test_PostScript.cc \
+ 	unit-tests/test_Ptr.cc \
+ 	unit-tests/test_Regex.cc \
  	unit-tests/test_Scalar.cc \
  	unit-tests/test_scoped_resource.cc \
 -	unit-tests/test_TextView.cc 
 +	unit-tests/test_TextView.cc
 +
 +if HAS_HKDF
 +test_tslib_SOURCES += \
 +	unit-tests/test_HKDF.cc
 +endif
  
  CompileParseRules_SOURCES = CompileParseRules.cc
  
diff --cc src/traffic_server/Makefile.inc
index c67b0d0,8175f78..dc7e7fe
--- a/src/traffic_server/Makefile.inc
+++ b/src/traffic_server/Makefile.inc
@@@ -97,8 -91,6 +92,12 @@@ traffic_server_traffic_server_LDADD = 
  	@YAMLCPP_LIBS@ \
  	-lm
  
+ if IS_DARWIN
+ traffic_server_traffic_server_LDADD += $(LUAJIT_DARWIN_LDFLAGS)
+ endif
++
 +if ENABLE_QUIC
 +traffic_server_traffic_server_LDADD += \
 +  $(top_builddir)/proxy/hq/libhq.a \
 +  $(top_builddir)/iocore/net/quic/libquic.a
 +endif