fix(tools/http): omit optional nil query parameters by duwenxin99 · Pull Request #1762 · googleapis/mcp-toolbox

duwenxin99 · 2025-10-21T21:21:08Z

🛠️ Fixes #1737

Yuan325

last nits~

…is#1762) 🛠️ Fixes googleapis#1737 bd16ba3

🤖 I have created a release *beep* *boop* --- ## [0.18.0](v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([#1770](#1770)) ([eaf7740](eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([#1673](#1673)) ([089081f](089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([#1707](#1707)) ([eeb694c](eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([#1717](#1717)) ([02f7f8a](02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([#1761](#1761)) ([a06d0d8](a06d0d8)) * **tools/http:** Omit optional nil query parameters ([#1762](#1762)) ([bd16ba3](bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([#1758](#1758)) ([336de1b](336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

🤖 I have created a release *beep* *boop* --- ## [0.18.0](v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([#1770](#1770)) ([eaf7740](eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([#1673](#1673)) ([089081f](089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([#1707](#1707)) ([eeb694c](eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([#1717](#1717)) ([02f7f8a](02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([#1761](#1761)) ([a06d0d8](a06d0d8)) * **tools/http:** Omit optional nil query parameters ([#1762](#1762)) ([bd16ba3](bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([#1758](#1758)) ([336de1b](336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> 3ca58b1

🤖 I have created a release *beep* *boop* --- ## [0.18.0](googleapis/mcp-toolbox@v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([googleapis#1770](googleapis#1770)) ([eaf7740](googleapis@eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([googleapis#1673](googleapis#1673)) ([089081f](googleapis@089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([googleapis#1707](googleapis#1707)) ([eeb694c](googleapis@eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([googleapis#1717](googleapis#1717)) ([02f7f8a](googleapis@02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([googleapis#1761](googleapis#1761)) ([a06d0d8](googleapis@a06d0d8)) * **tools/http:** Omit optional nil query parameters ([googleapis#1762](googleapis#1762)) ([bd16ba3](googleapis@bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([googleapis#1758](googleapis#1758)) ([336de1b](googleapis@336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com> 3ca58b1

🛠️ Fixes #1737

🤖 I have created a release *beep* *boop* --- ## [0.18.0](v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([#1770](#1770)) ([eaf7740](eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([#1673](#1673)) ([089081f](089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([#1707](#1707)) ([eeb694c](eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([#1717](#1717)) ([02f7f8a](02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([#1761](#1761)) ([a06d0d8](a06d0d8)) * **tools/http:** Omit optional nil query parameters ([#1762](#1762)) ([bd16ba3](bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([#1758](#1758)) ([336de1b](336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

🛠️ Fixes #1737

🤖 I have created a release *beep* *boop* --- ## [0.18.0](v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([#1770](#1770)) ([eaf7740](eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([#1673](#1673)) ([089081f](089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([#1707](#1707)) ([eeb694c](eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([#1717](#1717)) ([02f7f8a](02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([#1761](#1761)) ([a06d0d8](a06d0d8)) * **tools/http:** Omit optional nil query parameters ([#1762](#1762)) ([bd16ba3](bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([#1758](#1758)) ([336de1b](336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

🛠️ Fixes #1737

🤖 I have created a release *beep* *boop* --- ## [0.18.0](v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([#1770](#1770)) ([eaf7740](eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([#1673](#1673)) ([089081f](089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([#1707](#1707)) ([eeb694c](eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([#1717](#1717)) ([02f7f8a](02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([#1761](#1761)) ([a06d0d8](a06d0d8)) * **tools/http:** Omit optional nil query parameters ([#1762](#1762)) ([bd16ba3](bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([#1758](#1758)) ([336de1b](336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

🛠️ Fixes googleapis#1737

🤖 I have created a release *beep* *boop* --- ## [0.18.0](googleapis/mcp-toolbox@v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([googleapis#1770](googleapis#1770)) ([eaf7740](googleapis@eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([googleapis#1673](googleapis#1673)) ([089081f](googleapis@089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([googleapis#1707](googleapis#1707)) ([eeb694c](googleapis@eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([googleapis#1717](googleapis#1717)) ([02f7f8a](googleapis@02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([googleapis#1761](googleapis#1761)) ([a06d0d8](googleapis@a06d0d8)) * **tools/http:** Omit optional nil query parameters ([googleapis#1762](googleapis#1762)) ([bd16ba3](googleapis@bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([googleapis#1758](googleapis#1758)) ([336de1b](googleapis@336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

🤖 I have created a release *beep* *boop* --- ## [0.18.0](googleapis/mcp-toolbox@v0.17.0...v0.18.0) (2025-10-23) ### Features * Support `allowedValues`, `escape`, `minValue` and `maxValue` for parameters ([#1770](googleapis/mcp-toolbox#1770)) ([eaf7740](googleapis/mcp-toolbox@eaf7740)) * **tools/looker:** Tools to allow the agent to retrieve, create, modify, and delete LookML project files. ([#1673](googleapis/mcp-toolbox#1673)) ([089081f](googleapis/mcp-toolbox@089081f)) ### Bug Fixes * **sources/mysql:** Escape mysql user agent ([#1707](googleapis/mcp-toolbox#1707)) ([eeb694c](googleapis/mcp-toolbox@eeb694c)) * **sources/mysql:** Escape program_name for MySQL ([#1717](googleapis/mcp-toolbox#1717)) ([02f7f8a](googleapis/mcp-toolbox@02f7f8a)) * **tools/http:** Allow 2xx status code on tool invocation ([#1761](googleapis/mcp-toolbox#1761)) ([a06d0d8](googleapis/mcp-toolbox@a06d0d8)) * **tools/http:** Omit optional nil query parameters ([#1762](googleapis/mcp-toolbox#1762)) ([bd16ba3](googleapis/mcp-toolbox@bd16ba3)) * **tools/looker:** Looker file content calls should not use url.QueryEscape ([#1758](googleapis/mcp-toolbox#1758)) ([336de1b](googleapis/mcp-toolbox@336de1b)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). --------- Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com> Co-authored-by: Wenxin Du <117315983+duwenxin99@users.noreply.github.com>

duwenxin99 added 2 commits October 21, 2025 17:19

fix(tools/http): omit optional nil query parameters

d17386e

fix(tools/http): omit optional nil query parameters

333b1f5

duwenxin99 requested a review from a team October 21, 2025 21:21

blunderbuss-gcf Bot assigned Yuan325 Oct 21, 2025

rm

40dd398

duwenxin99 added the release candidate Use label to signal PR should be included in the next release. label Oct 21, 2025

duwenxin99 added 3 commits October 21, 2025 17:43

remove json encoding

d949056

fix test

ce9bea2

encode &

8abf08f

Yuan325 requested changes Oct 22, 2025

View reviewed changes

Comment thread tests/http/http_integration_test.go Outdated

Comment thread tests/http/http_integration_test.go

Comment thread tests/http/http_integration_test.go Outdated

Comment thread tests/http/http_integration_test.go Outdated

resolve comments

7ef5473

Yuan325 approved these changes Oct 22, 2025

View reviewed changes

Comment thread tests/http/http_integration_test.go Outdated

Comment thread tests/http/http_integration_test.go

duwenxin99 and others added 2 commits October 22, 2025 21:25

format

c350277

Merge branch 'main' into http-qp

ae604b6

duwenxin99 enabled auto-merge (squash) October 23, 2025 02:36

Merge branch 'main' into http-qp

e5665f1

duwenxin99 merged commit bd16ba3 into main Oct 23, 2025
11 checks passed

duwenxin99 deleted the http-qp branch October 23, 2025 02:56

release-please Bot mentioned this pull request Oct 23, 2025

chore(main): release 0.18.0 #1719

Merged

github-actions Bot pushed a commit to Jaleel-zhu/genai-toolbox that referenced this pull request Oct 23, 2025

deploy: fix(tools/http): omit optional nil query parameters (googleap…

325483e

…is#1762) 🛠️ Fixes googleapis#1737 bd16ba3

github-actions Bot pushed a commit to Jaleel-zhu/genai-toolbox that referenced this pull request Oct 23, 2025

deploy: fix(tools/http): omit optional nil query parameters (googleap…

cbd1341

…is#1762) 🛠️ Fixes googleapis#1737 bd16ba3

github-actions Bot pushed a commit to bhardwajRahul/genai-toolbox that referenced this pull request Oct 23, 2025

deploy: fix(tools/http): omit optional nil query parameters (googleap…

5a602e4

…is#1762) 🛠️ Fixes googleapis#1737 bd16ba3

github-actions Bot pushed a commit to bhardwajRahul/genai-toolbox that referenced this pull request Oct 23, 2025

deploy: fix(tools/http): omit optional nil query parameters (googleap…

19cd4b0

…is#1762) 🛠️ Fixes googleapis#1737 bd16ba3

BrewTestBot mentioned this pull request Oct 23, 2025

mcp-toolbox 0.18.0 Homebrew/homebrew-core#250869

Merged

twishabansal pushed a commit that referenced this pull request Oct 27, 2025

fix(tools/http): omit optional nil query parameters (#1762)

6993c94

🛠️ Fixes #1737

rahulpinto19 pushed a commit that referenced this pull request Oct 27, 2025

fix(tools/http): omit optional nil query parameters (#1762)

4c36a13

🛠️ Fixes #1737

rahulpinto19 pushed a commit that referenced this pull request Oct 27, 2025

fix(tools/http): omit optional nil query parameters (#1762)

3495c63

🛠️ Fixes #1737

srividyareddy786 pushed a commit to srividyareddy786/genai-toolbox that referenced this pull request Nov 4, 2025

fix(tools/http): omit optional nil query parameters (googleapis#1762)

4c4d253

🛠️ Fixes googleapis#1737

release-please Bot mentioned this pull request Nov 7, 2025

chore(main): release 0.19.1 #1901

Merged

github-actions Bot mentioned this pull request Feb 20, 2026

feat(tools/looker): Enable Get All Lookml Tests tool for Looker Anthrino/genai-toolbox#3

Open

6 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(tools/http): omit optional nil query parameters#1762

fix(tools/http): omit optional nil query parameters#1762
duwenxin99 merged 10 commits into
mainfrom
http-qp

duwenxin99 commented Oct 21, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Yuan325 left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

duwenxin99 commented Oct 21, 2025

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Yuan325 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants