feat: Add Tool authenticated parameters by duwenxin99 · Pull Request #80 · googleapis/mcp-toolbox

duwenxin99 · 2024-11-18T21:47:27Z

Add []ParamAuthSource to every Parameter type implementation to support authenticated configs. Create new constructors for types with auth.
Tool invocation API changes to parse auth header and authentecated parameters.
Add authSources to Tool manifest.

kurtisvg

broad note: I'd like to see more more involved testing here

Add auth params parsing implementation and tool calling API changes to support auth header.

1. OAuth provider test must run on Cloud Build to be able to print ID tokens of the GCE VM attached service account. 2. Borrowing gcloud CLI team's public `Client ID` in the Google ID token [documentation](https://cloud.google.com/docs/authentication/token-types#id-contents)

🤖 I have created a release *beep* *boop* --- ## [0.0.4](v0.0.3...v0.0.4) (2024-12-18) ### Features * Add `auth_required` to tools ([#123](#123)) ([3118104](3118104)) * Add Auth Source configuration ([#71](#71)) ([77b0d43](77b0d43)) * Add Tool authenticated parameters ([#80](#80)) ([380a6fb](380a6fb)) * **langchain-sdk:** Correctly parse Manifest API response as JSON ([#143](#143)) ([2c8633c](2c8633c)) * **langchain-sdk:** Support authentication in LangChain Toolbox SDK. ([#133](#133)) ([23fa912](23fa912)) ### Bug Fixes * Fix release image version tag ([#136](#136)) ([6d19ff9](6d19ff9)) * **langchain-sdk:** Correct test name to ensure execution and full coverage. ([#145](#145)) ([d820ac3](d820ac3)) * Set server version ([#150](#150)) ([abd1eb7](abd1eb7)) ### Miscellaneous Chores * Release 0.0.4 ([#152](#152)) ([86ec12f](86ec12f)) --- 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>

1. Add []ParamAuthSource to every Parameter type implementation to support authenticated configs. Create new constructors for types with auth. 2. Tool invocation API changes to parse auth header and authentecated parameters. 3. Add authSources to Tool manifest.

🤖 I have created a release *beep* *boop* --- ## [0.0.4](googleapis/mcp-toolbox@v0.0.3...v0.0.4) (2024-12-18) ### Features * Add `auth_required` to tools ([googleapis#123](googleapis#123)) ([3118104](googleapis@3118104)) * Add Auth Source configuration ([googleapis#71](googleapis#71)) ([77b0d43](googleapis@77b0d43)) * Add Tool authenticated parameters ([googleapis#80](googleapis#80)) ([380a6fb](googleapis@380a6fb)) * **langchain-sdk:** Correctly parse Manifest API response as JSON ([googleapis#143](googleapis#143)) ([2c8633c](googleapis@2c8633c)) * **langchain-sdk:** Support authentication in LangChain Toolbox SDK. ([googleapis#133](googleapis#133)) ([23fa912](googleapis@23fa912)) ### Bug Fixes * Fix release image version tag ([googleapis#136](googleapis#136)) ([6d19ff9](googleapis@6d19ff9)) * **langchain-sdk:** Correct test name to ensure execution and full coverage. ([googleapis#145](googleapis#145)) ([d820ac3](googleapis@d820ac3)) * Set server version ([googleapis#150](googleapis#150)) ([abd1eb7](googleapis@abd1eb7)) ### Miscellaneous Chores * Release 0.0.4 ([googleapis#152](googleapis#152)) ([86ec12f](googleapis@86ec12f)) --- 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>

🤖 I have created a release *beep* *boop* --- ## [0.0.4](googleapis/mcp-toolbox@v0.0.3...v0.0.4) (2024-12-18) ### Features * Add `auth_required` to tools ([#123](googleapis/mcp-toolbox#123)) ([3118104](googleapis/mcp-toolbox@3118104)) * Add Auth Source configuration ([#71](googleapis/mcp-toolbox#71)) ([77b0d43](googleapis/mcp-toolbox@77b0d43)) * Add Tool authenticated parameters ([#80](googleapis/mcp-toolbox#80)) ([380a6fb](googleapis/mcp-toolbox@380a6fb)) * **langchain-sdk:** Correctly parse Manifest API response as JSON ([#143](googleapis/mcp-toolbox#143)) ([2c8633c](googleapis/mcp-toolbox@2c8633c)) * **langchain-sdk:** Support authentication in LangChain Toolbox SDK. ([#133](googleapis/mcp-toolbox#133)) ([23fa912](googleapis/mcp-toolbox@23fa912)) ### Bug Fixes * Fix release image version tag ([#136](googleapis/mcp-toolbox#136)) ([6d19ff9](googleapis/mcp-toolbox@6d19ff9)) * **langchain-sdk:** Correct test name to ensure execution and full coverage. ([#145](googleapis/mcp-toolbox#145)) ([d820ac3](googleapis/mcp-toolbox@d820ac3)) * Set server version ([#150](googleapis/mcp-toolbox#150)) ([abd1eb7](googleapis/mcp-toolbox@abd1eb7)) ### Miscellaneous Chores * Release 0.0.4 ([#152](googleapis/mcp-toolbox#152)) ([86ec12f](googleapis/mcp-toolbox@86ec12f)) --- 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>

duwenxin99 requested a review from a team November 18, 2024 21:47

duwenxin99 changed the base branch from main to authsource November 18, 2024 21:58

duwenxin99 requested review from Yuan325 and kurtisvg November 19, 2024 17:22

duwenxin99 force-pushed the authsource branch from 0a31bd3 to af50794 Compare November 19, 2024 20:52

duwenxin99 force-pushed the params branch from a99f955 to 4beddd6 Compare November 19, 2024 21:02

duwenxin99 force-pushed the authsource branch from bc623e2 to 7313664 Compare November 19, 2024 22:27

duwenxin99 force-pushed the params branch from 4beddd6 to 1039c9e Compare November 19, 2024 22:51

kurtisvg requested changes Nov 22, 2024

View reviewed changes

Comment thread internal/tools/parameters.go Outdated

Comment thread internal/tools/parameters.go Outdated

Comment thread internal/tools/parameters_test.go Outdated

Comment thread internal/tools/postgressql/postgressql.go Outdated

duwenxin99 force-pushed the authsource branch from 3de7c34 to 8746ecc Compare November 26, 2024 15:40

duwenxin99 force-pushed the params branch from 1039c9e to dd7da55 Compare November 27, 2024 20:27

duwenxin99 requested a review from kurtisvg November 27, 2024 20:48

Yuan325 reviewed Dec 3, 2024

View reviewed changes

Comment thread cmd/root_test.go

kurtisvg requested changes Dec 5, 2024

View reviewed changes

kurtisvg approved these changes Dec 6, 2024

View reviewed changes

Comment thread internal/tools/parameters.go Outdated

Comment thread internal/tools/parameters.go Outdated

duwenxin99 force-pushed the authsource branch 4 times, most recently from 8746ecc to 1c00242 Compare December 9, 2024 20:11

duwenxin99 force-pushed the params branch from dbe3325 to 50c2b05 Compare December 9, 2024 20:29

duwenxin99 force-pushed the authsource branch from 0a94bf4 to 20529eb Compare December 13, 2024 17:13

Base automatically changed from authsource to main December 13, 2024 17:22

duwenxin99 force-pushed the params branch from 3047b9c to 0037d8e Compare December 13, 2024 20:58

duwenxin99 added 7 commits December 13, 2024 23:58

feat: Add Auth Source configs

f677ba1

update test

1d91795

add GetName()

91fda1a

add google auth implementation

25d3ca0

rename

0223487

feat: Add Auth Source configs

ff6665c

rename

dd762a0

duwenxin99 and others added 14 commits December 13, 2024 23:58

update test

d03b640

add tests

61e4caa

add more tests

5883056

nit

b818378

update yaml field name

84b0489

update error message

137eb30

delete auth required feature

586ed74

delete auth required feature

f83ba0b

resolve comments

fd6d22b

nit

d6d9fae

rebase

4639892

more rebase fix

650321d

nit

0175bd6

feat: add authenticated parameters to tools (#83)

6f92c38

Add auth params parsing implementation and tool calling API changes to support auth header.

duwenxin99 changed the title ~~feat: Add AuthSources to Tool parameters~~ feat: Add Tool authenticated parameters Dec 14, 2024

duwenxin99 force-pushed the params branch from 9d73c73 to 6f92c38 Compare December 14, 2024 04:59

duwenxin99 and others added 3 commits December 15, 2024 22:33

update auth source for loop logics

ec56fce

Merge branch 'main' into params

fb1449c

duwenxin99 merged commit 380a6fb into main Dec 16, 2024

duwenxin99 deleted the params branch December 16, 2024 22:46

release-please Bot mentioned this pull request Dec 18, 2024

chore(main): release 0.0.4 #153

Merged

This was referenced Feb 18, 2026

Link Checker Report #2508

Closed

Link Checker Report #2513

Closed

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

Open

github-actions Bot mentioned this pull request Mar 12, 2026

chore(main): release 0.29.0 #2608

Merged

Navigation Menu

feat: Add Tool authenticated parameters - #80

feat: Add Tool authenticated parameters#80
duwenxin99 merged 26 commits into
mainfrom
params

duwenxin99 commented Nov 18, 2024 •

edited

Loading

Uh oh!

kurtisvg left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

duwenxin99 commented Nov 18, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

kurtisvg left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

duwenxin99 commented Nov 18, 2024 •

edited

Loading