Skip to content

fix: make header names python-safe#592

Open
saurabhkoshatwar wants to merge 5 commits into
googleapis:mainfrom
saurabhkoshatwar:skoshatwar/issue-fix-580
Open

fix: make header names python-safe#592
saurabhkoshatwar wants to merge 5 commits into
googleapis:mainfrom
saurabhkoshatwar:skoshatwar/issue-fix-580

Merge branch 'main' into skoshatwar/issue-fix-580

ad17db8
Select commit
Loading
Failed to load commit list.
Google Cloud Build / core-python-sdk-pr-py311 (toolbox-testing-438616) succeeded Apr 14, 2026 in 1m 8s

Summary

Build Information

Trigger core-python-sdk-pr-py311
Build 1375b6f3-162c-44ec-805a-3391ba81189a
Start 2026-04-13T22:18:13-07:00
Duration 1m4.863s
Status SUCCESS

Steps

Step Status Duration
Install requirements SUCCESS 36.947s
Run integration tests SUCCESS 23.063s

Details


starting build "1375b6f3-162c-44ec-805a-3391ba81189a"

FETCHSOURCE
From https://github.com/googleapis/mcp-toolbox-sdk-python
 * branch            ad17db87f56e59c21a0a071fb8cff2d65994ff9b -> FETCH_HEAD
HEAD is now at ad17db8 Merge branch 'main' into skoshatwar/issue-fix-580
GitCommit:
ad17db87f56e59c21a0a071fb8cff2d65994ff9b
BUILD
Starting Step #0 - "Install requirements"
Step #0 - "Install requirements": Pulling image: python:3.11
Step #0 - "Install requirements": 3.11: Pulling from library/python
Step #0 - "Install requirements": a7730063fcfe: Pulling fs layer
Step #0 - "Install requirements": f33970743aee: Pulling fs layer
Step #0 - "Install requirements": 5397da1d1537: Pulling fs layer
Step #0 - "Install requirements": 1ecbddc58afb: Pulling fs layer
Step #0 - "Install requirements": c30fe4e5a0d6: Pulling fs layer
Step #0 - "Install requirements": 0b4f6d60dcda: Pulling fs layer
Step #0 - "Install requirements": 4e8ce30c1c4c: Pulling fs layer
Step #0 - "Install requirements": 4e8ce30c1c4c: Waiting
Step #0 - "Install requirements": c30fe4e5a0d6: Verifying Checksum
Step #0 - "Install requirements": c30fe4e5a0d6: Download complete
Step #0 - "Install requirements": 4e8ce30c1c4c: Verifying Checksum
Step #0 - "Install requirements": 4e8ce30c1c4c: Download complete
Step #0 - "Install requirements": 0b4f6d60dcda: Verifying Checksum
Step #0 - "Install requirements": 0b4f6d60dcda: Download complete
Step #0 - "Install requirements": f33970743aee: Verifying Checksum
Step #0 - "Install requirements": f33970743aee: Download complete
Step #0 - "Install requirements": a7730063fcfe: Verifying Checksum
Step #0 - "Install requirements": a7730063fcfe: Download complete
Step #0 - "Install requirements": 5397da1d1537: Verifying Checksum
Step #0 - "Install requirements": 5397da1d1537: Download complete
Step #0 - "Install requirements": 1ecbddc58afb: Verifying Checksum
Step #0 - "Install requirements": 1ecbddc58afb: Download complete
Step #0 - "Install requirements": a7730063fcfe: Pull complete
Step #0 - "Install requirements": f33970743aee: Pull complete
Step #0 - "Install requirements": 5397da1d1537: Pull complete
Step #0 - "Install requirements": 1ecbddc58afb: Pull complete
Step #0 - "Install requirements": c30fe4e5a0d6: Pull complete
Step #0 - "Install requirements": 0b4f6d60dcda: Pull complete
Step #0 - "Install requirements": 4e8ce30c1c4c: Pull complete
Step #0 - "Install requirements": Digest: sha256:8f004bb4a5d9e8107f9d0a1aae78b990a6bf107b22888eda9c44ebccc42cd6cd
Step #0 - "Install requirements": Status: Downloaded newer image for python:3.11
Step #0 - "Install requirements": docker.io/library/python:3.11
Step #0 - "Install requirements": Collecting uv
Step #0 - "Install requirements":   Downloading uv-0.11.6-py3-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl.metadata (11 kB)
Step #0 - "Install requirements": Downloading uv-0.11.6-py3-none-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (25.0 MB)
Step #0 - "Install requirements":    ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 25.0/25.0 MB 79.7 MB/s eta 0:00:00
Step #0 - "Install requirements": Installing collected packages: uv
Step #0 - "Install requirements": Successfully installed uv-0.11.6
Step #0 - "Install requirements": WARNING: Running pip as the 'root' user can result in broken permissions and conflicting behaviour with the system package manager. It is recommended to use a virtual environment instead: https://pip.pypa.io/warnings/venv
Step #0 - "Install requirements": 
Step #0 - "Install requirements": [notice] A new release of pip is available: 24.0 -> 26.0.1
Step #0 - "Install requirements": [notice] To update, run: pip install --upgrade pip
Step #0 - "Install requirements": Using CPython 3.11.15 interpreter at: /usr/local/bin/python3
Step #0 - "Install requirements": Creating virtual environment at: /workspace/venv
Step #0 - "Install requirements": Activate with: source /workspace/venv/bin/activate
Step #0 - "Install requirements": Using Python 3.11.15 environment at: /workspace/venv
Step #0 - "Install requirements": Resolved 1 package in 257ms
Step #0 - "Install requirements": Downloading uv (23.8MiB)
Step #0 - "Install requirements":  Downloaded uv
Step #0 - "Install requirements": Prepared 1 package in 368ms
Step #0 - "Install requirements": warning: Failed to hardlink files; falling back to full copy. This may lead to degraded performance.
Step #0 - "Install requirements":          If the cache and target directories are on different filesystems, hardlinking may not be supported.
Step #0 - "Install requirements":          If this is intentional, set `export UV_LINK_MODE=copy` or use `--link-mode=copy` to suppress this warning.
Step #0 - "Install requirements": Installed 1 package in 58ms
Step #0 - "Install requirements":  + uv==0.11.6
Step #0 - "Install requirements": Using Python 3.11.15 environment at: /workspace/venv
Step #0 - "Install requirements": Resolved 85 packages in 2.15s
Step #0 - "Install requirements":    Building toolbox-core @ file:///workspace/packages/toolbox-core
Step #0 - "Install requirements": Downloading pydantic-core (2.0MiB)
Step #0 - "Install requirements": Downloading pygments (1.2MiB)
Step #0 - "Install requirements": Downloading aiohttp (1.7MiB)
Step #0 - "Install requirements": Downloading jedi (1.5MiB)
Step #0 - "Install requirements": Downloading black (1.7MiB)
Step #0 - "Install requirements": Downloading cryptography (4.3MiB)
Step #0 - "Install requirements": Downloading grpcio (6.5MiB)
Step #0 - "Install requirements": Downloading mypy (13.9MiB)
Step #0 - "Install requirements":       Built toolbox-core @ file:///workspace/packages/toolbox-core
Step #0 - "Install requirements":  Downloaded pydantic-core
Step #0 - "Install requirements":  Downloaded aiohttp
Step #0 - "Install requirements":  Downloaded grpcio
Step #0 - "Install requirements":  Downloaded black
Step #0 - "Install requirements":  Downloaded cryptography
Step #0 - "Install requirements":  Downloaded pygments
Step #0 - "Install requirements":  Downloaded mypy
Step #0 - "Install requirements":  Downloaded jedi
Step #0 - "Install requirements": Prepared 85 packages in 1.28s
Step #0 - "Install requirements": warning: Failed to hardlink files; falling back to full copy. This may lead to degraded performance.
Step #0 - "Install requirements":          If the cache and target directories are on different filesystems, hardlinking may not be supported.
Step #0 - "Install requirements":          If this is intentional, set `export UV_LINK_MODE=copy` or use `--link-mode=copy` to suppress this warning.
Step #0 - "Install requirements": Installed 85 packages in 621ms
Step #0 - "Install requirements":  + aiohappyeyeballs==2.6.1
Step #0 - "Install requirements":  + aiohttp==3.13.5
Step #0 - "Install requirements":  + aioresponses==0.7.8
Step #0 - "Install requirements":  + aiosignal==1.4.0
Step #0 - "Install requirements":  + annotated-types==0.7.0
Step #0 - "Install requirements":  + asttokens==3.0.1
Step #0 - "Install requirements":  + attrs==26.1.0
Step #0 - "Install requirements":  + black==26.3.1
Step #0 - "Install requirements":  + certifi==2026.2.25
Step #0 - "Install requirements":  + cffi==2.0.0
Step #0 - "Install requirements":  + charset-normalizer==3.4.7
Step #0 - "Install requirements":  + click==8.3.2
Step #0 - "Install requirements":  + coverage==7.13.5
Step #0 - "Install requirements":  + cryptography==46.0.7
Step #0 - "Install requirements":  + decorator==5.2.1
Step #0 - "Install requirements":  + deprecated==1.3.1
Step #0 - "Install requirements":  + executing==2.2.1
Step #0 - "Install requirements":  + frozenlist==1.8.0
Step #0 - "Install requirements":  + google-api-core==2.30.3
Step #0 - "Install requirements":  + google-auth==2.49.2
Step #0 - "Install requirements":  + google-cloud-core==2.5.1
Step #0 - "Install requirements":  + google-cloud-secret-manager==2.27.0
Step #0 - "Install requirements":  + google-cloud-storage==3.10.1
Step #0 - "Install requirements":  + google-crc32c==1.8.0
Step #0 - "Install requirements":  + google-resumable-media==2.8.2
Step #0 - "Install requirements":  + googleapis-common-protos==1.74.0
Step #0 - "Install requirements":  + grpc-google-iam-v1==0.14.4
Step #0 - "Install requirements":  + grpcio==1.80.0
Step #0 - "Install requirements":  + grpcio-status==1.80.0
Step #0 - "Install requirements":  + idna==3.11
Step #0 - "Install requirements":  + importlib-metadata==8.7.1
Step #0 - "Install requirements":  + iniconfig==2.3.0
Step #0 - "Install requirements":  + ipython==9.10.1
Step #0 - "Install requirements":  + ipython-pygments-lexers==1.1.1
Step #0 - "Install requirements":  + isort==8.0.1
Step #0 - "Install requirements":  + jedi==0.19.2
Step #0 - "Install requirements":  + librt==0.9.0
Step #0 - "Install requirements":  + matplotlib-inline==0.2.1
Step #0 - "Install requirements":  + multidict==6.7.1
Step #0 - "Install requirements":  + mypy==1.20.0
Step #0 - "Install requirements":  + mypy-extensions==1.1.0
Step #0 - "Install requirements":  + opentelemetry-api==1.38.0
Step #0 - "Install requirements":  + opentelemetry-exporter-otlp==1.38.0
Step #0 - "Install requirements":  + opentelemetry-exporter-otlp-proto-common==1.38.0
Step #0 - "Install requirements":  + opentelemetry-exporter-otlp-proto-grpc==1.38.0
Step #0 - "Install requirements":  + opentelemetry-exporter-otlp-proto-http==1.38.0
Step #0 - "Install requirements":  + opentelemetry-proto==1.38.0
Step #0 - "Install requirements":  + opentelemetry-sdk==1.38.0
Step #0 - "Install requirements":  + opentelemetry-semantic-conventions==0.59b0
Step #0 - "Install requirements":  + packaging==26.0
Step #0 - "Install requirements":  + parso==0.8.6
Step #0 - "Install requirements":  + pathspec==1.0.4
Step #0 - "Install requirements":  + pexpect==4.9.0
Step #0 - "Install requirements":  + platformdirs==4.9.6
Step #0 - "Install requirements":  + pluggy==1.6.0
Step #0 - "Install requirements":  + prompt-toolkit==3.0.52
Step #0 - "Install requirements":  + propcache==0.4.1
Step #0 - "Install requirements":  + proto-plus==1.27.2
Step #0 - "Install requirements":  + protobuf==6.33.6
Step #0 - "Install requirements":  + ptyprocess==0.7.0
Step #0 - "Install requirements":  + pure-eval==0.2.3
Step #0 - "Install requirements":  + pyasn1==0.6.3
Step #0 - "Install requirements":  + pyasn1-modules==0.4.2
Step #0 - "Install requirements":  + pycparser==3.0
Step #0 - "Install requirements":  + pydantic==2.13.0
Step #0 - "Install requirements":  + pydantic-core==2.46.0
Step #0 - "Install requirements":  + pygments==2.20.0
Step #0 - "Install requirements":  + pytest==9.0.2
Step #0 - "Install requirements":  + pytest-aioresponses==0.3.0
Step #0 - "Install requirements":  + pytest-asyncio==1.3.0
Step #0 - "Install requirements":  + pytest-cov==7.1.0
Step #0 - "Install requirements":  + pytest-mock==3.15.1
Step #0 - "Install requirements":  + pytokens==0.4.1
Step #0 - "Install requirements":  + requests==2.33.1
Step #0 - "Install requirements":  + stack-data==0.6.3
Step #0 - "Install requirements":  + tokenize-rt==6.2.0
Step #0 - "Install requirements":  + toolbox-core==1.0.0 (from file:///workspace/packages/toolbox-core)
Step #0 - "Install requirements":  + traitlets==5.14.3
Step #0 - "Install requirements":  + typing-extensions==4.15.0
Step #0 - "Install requirements":  + typing-inspection==0.4.2
Step #0 - "Install requirements":  + urllib3==2.6.3
Step #0 - "Install requirements":  + wcwidth==0.6.0
Step #0 - "Install requirements":  + wrapt==2.1.2
Step #0 - "Install requirements":  + yarl==1.23.0
Step #0 - "Install requirements":  + zipp==3.23.1
Step #0 - "Install requirements": Using Python 3.11.15 environment at: /workspace/venv
Step #0 - "Install requirements": Resolved 26 packages in 160ms
Step #0 - "Install requirements": Downloading pydantic-core (2.0MiB)
Step #0 - "Install requirements":  Downloaded pydantic-core
Step #0 - "Install requirements": Prepared 3 packages in 95ms
Step #0 - "Install requirements": Uninstalled 3 packages in 5ms
Step #0 - "Install requirements": warning: Failed to hardlink files; falling back to full copy. This may lead to degraded performance.
Step #0 - "Install requirements":          If the cache and target directories are on different filesystems, hardlinking may not be supported.
Step #0 - "Install requirements":          If this is intentional, set `export UV_LINK_MODE=copy` or use `--link-mode=copy` to suppress this warning.
Step #0 - "Install requirements": Installed 3 packages in 23ms
Step #0 - "Install requirements":  - google-auth==2.49.2
Step #0 - "Install requirements":  + google-auth==2.49.1
Step #0 - "Install requirements":  - pydantic==2.13.0
Step #0 - "Install requirements":  + pydantic==2.12.5
Step #0 - "Install requirements":  - pydantic-core==2.46.0
Step #0 - "Install requirements":  + pydantic-core==2.41.5
Finished Step #0 - "Install requirements"
Starting Step #1 - "Run integration tests"
Step #1 - "Run integration tests": Already have image (with digest): python:3.11
Step #1 - "Run integration tests": ============================= test session starts ==============================
Step #1 - "Run integration tests": platform linux -- Python 3.11.15, pytest-9.0.2, pluggy-1.6.0
Step #1 - "Run integration tests": rootdir: /workspace/packages/toolbox-core
Step #1 - "Run integration tests": configfile: pyproject.toml
Step #1 - "Run integration tests": plugins: cov-7.1.0, asyncio-1.3.0, aioresponses-0.3.0, mock-3.15.1
Step #1 - "Run integration tests": asyncio: mode=Mode.STRICT, debug=False, asyncio_default_fixture_loop_scope=None, asyncio_default_test_loop_scope=function
Step #1 - "Run integration tests": collected 527 items
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": tests/mcp_transport/test_base.py .............                           [  2%]
Step #1 - "Run integration tests": tests/mcp_transport/test_telemetry.py .................................. [  8%]
Step #1 - "Run integration tests": ...........                                                              [ 11%]
Step #1 - "Run integration tests": tests/mcp_transport/test_v20241105.py .................................. [ 17%]
Step #1 - "Run integration tests":                                                                          [ 17%]
Step #1 - "Run integration tests": tests/mcp_transport/test_v20250326.py .................................. [ 23%]
Step #1 - "Run integration tests":                                                                          [ 23%]
Step #1 - "Run integration tests": tests/mcp_transport/test_v20250618.py .................................. [ 30%]
Step #1 - "Run integration tests": ..                                                                       [ 30%]
Step #1 - "Run integration tests": tests/mcp_transport/test_v20251125.py .................................. [ 37%]
Step #1 - "Run integration tests": ..                                                                       [ 37%]
Step #1 - "Run integration tests": tests/test_auth_methods.py .........                                     [ 39%]
Step #1 - "Run integration tests": tests/test_client.py .........................                           [ 44%]
Step #1 - "Run integration tests": tests/test_e2e.py ...................................                    [ 50%]
Step #1 - "Run integration tests": tests/test_e2e_mcp.py .................................................. [ 60%]
Step #1 - "Run integration tests": ........................................................................ [ 73%]
Step #1 - "Run integration tests": ..........                                                               [ 75%]
Step #1 - "Run integration tests": tests/test_protocol.py ....................                              [ 79%]
Step #1 - "Run integration tests": tests/test_sync_client.py ................                               [ 82%]
Step #1 - "Run integration tests": tests/test_sync_e2e.py ..............                                    [ 85%]
Step #1 - "Run integration tests": tests/test_sync_tool.py ...................                              [ 88%]
Step #1 - "Run integration tests": tests/test_tool.py ..............................                        [ 94%]
Step #1 - "Run integration tests": tests/test_utils.py .............................                        [100%]
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": =============================== warnings summary ===============================
Step #1 - "Run integration tests": tests/test_tool.py:567
Step #1 - "Run integration tests":   /workspace/packages/toolbox-core/tests/test_tool.py:567: DeprecationWarning: invalid escape sequence '\('
Step #1 - "Run integration tests":     expected_error_message = "Authentication source\(s\) \`unused-auth-service\` unused by tool \`sample_tool\`."
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": tests/test_client.py: 9 warnings
Step #1 - "Run integration tests": tests/test_e2e.py: 6 warnings
Step #1 - "Run integration tests": tests/test_e2e_mcp.py: 24 warnings
Step #1 - "Run integration tests": tests/test_sync_client.py: 4 warnings
Step #1 - "Run integration tests": tests/test_sync_e2e.py: 5 warnings
Step #1 - "Run integration tests":   /workspace/packages/toolbox-core/src/toolbox_core/utils.py:56: UserWarning: This connection is using HTTP. To prevent credential exposure, please ensure all communication is sent over HTTPS.
Step #1 - "Run integration tests":     warnings.warn(
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": tests/test_sync_client.py::TestSyncClientLifecycle::test_load_tool_raises_if_loop_or_thread_none
Step #1 - "Run integration tests":   /workspace/packages/toolbox-core/tests/test_sync_client.py:333: RuntimeWarning: coroutine 'ToolboxClient.load_tool' was never awaited
Step #1 - "Run integration tests":     with pytest.raises(
Step #1 - "Run integration tests":   Enable tracemalloc to get traceback where the object was allocated.
Step #1 - "Run integration tests":   See https://docs.pytest.org/en/stable/how-to/capture-warnings.html#resource-warnings for more info.
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": tests/test_sync_client.py::TestSyncClientLifecycle::test_load_tool_raises_if_loop_or_thread_none
Step #1 - "Run integration tests":   /workspace/packages/toolbox-core/tests/test_sync_client.py:342: RuntimeWarning: coroutine 'ToolboxClient.load_toolset' was never awaited
Step #1 - "Run integration tests":     with pytest.raises(
Step #1 - "Run integration tests":   Enable tracemalloc to get traceback where the object was allocated.
Step #1 - "Run integration tests":   See https://docs.pytest.org/en/stable/how-to/capture-warnings.html#resource-warnings for more info.
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": -- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
Step #1 - "Run integration tests": ================================ tests coverage ================================
Step #1 - "Run integration tests": _______________ coverage: platform linux, python 3.11.15-final-0 _______________
Step #1 - "Run integration tests": 
Step #1 - "Run integration tests": Name                                                Stmts   Miss  Cover
Step #1 - "Run integration tests": -----------------------------------------------------------------------
Step #1 - "Run integration tests": src/toolbox_core/__init__.py                            3      0   100%
Step #1 - "Run integration tests": src/toolbox_core/auth_methods.py                       58      2    97%
Step #1 - "Run integration tests": src/toolbox_core/client.py                             89      2    98%
Step #1 - "Run integration tests": src/toolbox_core/itransport.py                         20      5    75%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/__init__.py              5      0   100%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/telemetry.py           153     14    91%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/transport_base.py       97      3    97%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20241105/mcp.py       122     12    90%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20241105/types.py      94      1    99%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20250326/mcp.py       135     16    88%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20250326/types.py      94      1    99%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20250618/mcp.py       125     12    90%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20250618/types.py      94      1    99%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20251125/mcp.py       125     12    90%
Step #1 - "Run integration tests": src/toolbox_core/mcp_transport/v20251125/types.py      94      1    99%
Step #1 - "Run integration tests": src/toolbox_core/protocol.py                           72      4    94%
Step #1 - "Run integration tests": src/toolbox_core/sync_client.py                        45      0   100%
Step #1 - "Run integration tests": src/toolbox_core/sync_tool.py                          64      0   100%
Step #1 - "Run integration tests": src/toolbox_core/tool.py                              116      7    94%
Step #1 - "Run integration tests": src/toolbox_core/utils.py                              62      0   100%
Step #1 - "Run integration tests": src/toolbox_core/version.py                             1      0   100%
Step #1 - "Run integration tests": -----------------------------------------------------------------------
Step #1 - "Run integration tests": TOTAL                                                1668     93    94%
Step #1 - "Run integration tests": Required test coverage of 90% reached. Total coverage: 94.42%
Step #1 - "Run integration tests": ====================== 527 passed, 51 warnings in 16.90s =======================
Finished Step #1 - "Run integration tests"
PUSH
DONE

Build Log: https://console.cloud.google.com/logs/viewer?advancedFilter=resource.type%3D%22build%22+AND+resource.labels.build_id%3D%221375b6f3-162c-44ec-805a-3391ba81189a%22+AND+resource.labels.build_trigger_id%3D%228ead8fc9-3a20-4cbd-aa08-019e44acf22d%22&project=toolbox-testing-438616