You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bn...@apache.org on 2021/04/26 21:19:03 UTC

[trafficserver] branch master updated: Add a chunked disabled test. (#7743)

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

bneradt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 60e291d  Add a chunked disabled test. (#7743)
60e291d is described below

commit 60e291d0b84f8020256f5790cb7d9f94f8199364
Author: Brian Neradt <br...@gmail.com>
AuthorDate: Mon Apr 26 16:18:53 2021 -0500

    Add a chunked disabled test. (#7743)
    
    This adds a "proxy.config.http.chunking_enabled: 0" test in which the
    server responds with a chunked response, both for HTTP and HTTPS.
---
 .../chunked_encoding_disabled.test.py              | 72 +++++++++++++++++++
 .../gold/verifier_client_chunked.gold              | 13 ++++
 .../chunked_encoding/replays/chunked.replay.yaml   | 84 ++++++++++++++++++++++
 3 files changed, 169 insertions(+)

diff --git a/tests/gold_tests/chunked_encoding/chunked_encoding_disabled.test.py b/tests/gold_tests/chunked_encoding/chunked_encoding_disabled.test.py
new file mode 100644
index 0000000..6cf2894
--- /dev/null
+++ b/tests/gold_tests/chunked_encoding/chunked_encoding_disabled.test.py
@@ -0,0 +1,72 @@
+'''
+Test ATS configured to not respond with chunked content.
+'''
+#  Licensed to the Apache Software Foundation (ASF) under one
+#  or more contributor license agreements.  See the NOTICE file
+#  distributed with this work for additional information
+#  regarding copyright ownership.  The ASF licenses this file
+#  to you under the Apache License, Version 2.0 (the
+#  "License"); you may not use this file except in compliance
+#  with the License.  You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+
+Test.Summary = 'Test ATS configured to not respond with chunked content.'
+
+
+class ChunkedEncodingDisabled:
+    chunkedReplayFile = "replays/chunked.replay.yaml"
+
+    def __init__(self):
+        self.setupOriginServer()
+        self.setupTS()
+
+    def setupOriginServer(self):
+        self.server = Test.MakeVerifierServerProcess("server", self.chunkedReplayFile)
+
+    def setupTS(self):
+        self.ts = Test.MakeATSProcess("ts", enable_tls=True, enable_cache=False)
+        self.ts.addDefaultSSLFiles()
+        self.ts.Disk.records_config.update({
+            "proxy.config.diags.debug.enabled": 1,
+            "proxy.config.diags.debug.tags": "http",
+            "proxy.config.ssl.server.cert.path": f'{self.ts.Variables.SSLDir}',
+            "proxy.config.ssl.server.private_key.path": f'{self.ts.Variables.SSLDir}',
+            "proxy.config.ssl.client.verify.server.policy": 'PERMISSIVE',
+
+            # Never respond with chunked encoding.
+            "proxy.config.http.chunking_enabled": 0,
+        })
+        self.ts.Disk.ssl_multicert_config.AddLine(
+            'dest_ip=* ssl_cert_name=server.pem ssl_key_name=server.key'
+        )
+        self.ts.Disk.remap_config.AddLines([
+            f"map /for/http http://127.0.0.1:{self.server.Variables.http_port}/",
+            f"map /for/tls https://127.0.0.1:{self.server.Variables.https_port}/",
+        ])
+
+    def runChunkedTraffic(self):
+        tr = Test.AddTestRun()
+        tr.AddVerifierClientProcess(
+            "client",
+            self.chunkedReplayFile,
+            http_ports=[self.ts.Variables.port],
+            https_ports=[self.ts.Variables.ssl_port])
+        tr.Processes.Default.Streams.stdout += "gold/verifier_client_chunked.gold"
+
+        tr.Processes.Default.StartBefore(self.server)
+        tr.Processes.Default.StartBefore(self.ts)
+        tr.StillRunningAfter = self.server
+        tr.StillRunningAfter = self.ts
+
+    def run(self):
+        self.runChunkedTraffic()
+
+
+ChunkedEncodingDisabled().run()
diff --git a/tests/gold_tests/chunked_encoding/gold/verifier_client_chunked.gold b/tests/gold_tests/chunked_encoding/gold/verifier_client_chunked.gold
new file mode 100644
index 0000000..f2beb34
--- /dev/null
+++ b/tests/gold_tests/chunked_encoding/gold/verifier_client_chunked.gold
@@ -0,0 +1,13 @@
+``
+- "connection": "keep-alive"
+``
+- "Connection": "close"
+``
+``content: 0000000 0000001 0000002 0000003 
+``
+- "connection": "keep-alive"
+``
+- "Connection": "close"
+``
+``content: 0000000 0000001 0000002 0000003 
+``
diff --git a/tests/gold_tests/chunked_encoding/replays/chunked.replay.yaml b/tests/gold_tests/chunked_encoding/replays/chunked.replay.yaml
new file mode 100644
index 0000000..66c0c24
--- /dev/null
+++ b/tests/gold_tests/chunked_encoding/replays/chunked.replay.yaml
@@ -0,0 +1,84 @@
+#  Licensed to the Apache Software Foundation (ASF) under one
+#  or more contributor license agreements.  See the NOTICE file
+#  distributed with this work for additional information
+#  regarding copyright ownership.  The ASF licenses this file
+#  to you under the Apache License, Version 2.0 (the
+#  "License"); you may not use this file except in compliance
+#  with the License.  You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+#  Unless required by applicable law or agreed to in writing, software
+#  distributed under the License is distributed on an "AS IS" BASIS,
+#  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+#  See the License for the specific language governing permissions and
+#  limitations under the License.
+
+meta:
+  version: "1.0"
+
+sessions:
+- transactions:
+  - client-request:
+      method: "GET"
+      version: "1.1"
+      url: /for/http
+      headers:
+        fields:
+        - [ Host, example.com ]
+        - [ Connection, keep-alive ]
+        - [ uuid, 1 ]
+
+    server-response:
+      status: 200
+      reason: OK
+      headers:
+        fields:
+        - [ Transfer-Encoding, chunked ]
+        - [ Connection, keep-alive ]
+      content:
+        size: 32
+
+    proxy-response:
+      status: 200
+      headers:
+        fields:
+        - [ Transfer-Encoding, { as: absent } ]
+      content:
+        size: 32
+
+- protocol:
+  - name: http
+    version: 1
+  - name: tls
+  - name: tcp
+  - name: ip
+
+  transactions:
+  - client-request:
+      method: "GET"
+      version: "1.1"
+      url: /for/tls
+      headers:
+        fields:
+        - [ Host, example.com ]
+        - [ Connection, keep-alive ]
+        - [ uuid, 2 ]
+
+    server-response:
+      status: 200
+      reason: OK
+      headers:
+        fields:
+        - [ Transfer-Encoding, chunked ]
+        - [ Connection, keep-alive ]
+      content:
+        size: 32
+
+    proxy-response:
+      status: 200
+      headers:
+        fields:
+        - [ Transfer-Encoding, { as: absent } ]
+      content:
+        size: 32