From 8da22b15b9b8b3799ed457d877d261c68c92ecfd Mon Sep 17 00:00:00 2001 From: Diego Hurtado Date: Tue, 9 Feb 2021 16:54:43 -0600 Subject: [PATCH] Fix checks --- .github/workflows/test.yml | 2 +- .../opentelemetry/propagators/ot_trace/__init__.py | 12 ++++++------ .../tests/test_ot_trace_propagator.py | 4 ++-- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 9ee13517bc..4ff2dcdeb5 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -6,7 +6,7 @@ on: - 'release/*' pull_request: env: - CORE_REPO_SHA: bcf15a417c3f029e1e75ec824925bc02808d21eb + CORE_REPO_SHA: d151f5689874ce78ddedc9aa6051be28078d7de8 jobs: build: diff --git a/propagator/opentelemetry-propagator-ot-trace/src/opentelemetry/propagators/ot_trace/__init__.py b/propagator/opentelemetry-propagator-ot-trace/src/opentelemetry/propagators/ot_trace/__init__.py index 728b938507..b7e9cf5449 100644 --- a/propagator/opentelemetry-propagator-ot-trace/src/opentelemetry/propagators/ot_trace/__init__.py +++ b/propagator/opentelemetry-propagator-ot-trace/src/opentelemetry/propagators/ot_trace/__init__.py @@ -38,8 +38,8 @@ OT_SAMPLED_HEADER = "ot-tracer-sampled" OT_BAGGAGE_PREFIX = "ot-baggage-" -_valid_header_name = re_compile(r"^[\w_^`!#$%&'*+.|~]+$") -_valid_header_value = re_compile(r"^[\t\x20-\x7e\x80-\xff]+$") +_valid_header_name = re_compile(r"[\w_^`!#$%&'*+.|~]+") +_valid_header_value = re_compile(r"[\t\x20-\x7e\x80-\xff]+") _valid_extract_traceid = re_compile(r"[0-9a-f]{1,32}") _valid_extract_spanid = re_compile(r"[0-9a-f]{1,16}") @@ -71,9 +71,9 @@ def extract( if ( traceid != INVALID_TRACE_ID - and _valid_extract_traceid.match(traceid) is not None + and _valid_extract_traceid.fullmatch(traceid) is not None and spanid != INVALID_SPAN_ID - and _valid_extract_spanid.match(spanid) is not None + and _valid_extract_spanid.fullmatch(spanid) is not None ): context = set_span_in_context( DefaultSpan( @@ -137,8 +137,8 @@ def inject( for header_name, header_value in baggage.items(): if ( - _valid_header_name.match(header_name) is None - or _valid_header_value.match(header_value) is None + _valid_header_name.fullmatch(header_name) is None + or _valid_header_value.fullmatch(header_value) is None ): continue diff --git a/propagator/opentelemetry-propagator-ot-trace/tests/test_ot_trace_propagator.py b/propagator/opentelemetry-propagator-ot-trace/tests/test_ot_trace_propagator.py index f7c811185a..3c4cd7c166 100644 --- a/propagator/opentelemetry-propagator-ot-trace/tests/test_ot_trace_propagator.py +++ b/propagator/opentelemetry-propagator-ot-trace/tests/test_ot_trace_propagator.py @@ -285,7 +285,7 @@ def test_extract_malformed_trace_id(self): self.ot_trace_propagator.extract( carrier_getter, { - OT_TRACE_ID_HEADER: "abc123", + OT_TRACE_ID_HEADER: "abc123!", OT_SPAN_ID_HEADER: "e457b5a2e4d86bd1", OT_SAMPLED_HEADER: "false", }, @@ -302,7 +302,7 @@ def test_extract_malformed_span_id(self): carrier_getter, { OT_TRACE_ID_HEADER: "64fe8b2a57d3eff7", - OT_SPAN_ID_HEADER: "abc123", + OT_SPAN_ID_HEADER: "abc123!", OT_SAMPLED_HEADER: "false", }, )