oauth client example: skip DCR if MCP_CLIENT_ID is set (#549)

djoreilly · web-flow · commit 5298f646e70e · 2025-09-02T10:56:09.000+03:00
If the user has set the MCP_CLIENT_ID environmental variable then they
don't want Dynamic Client Registration and it should be skipped.
diff --git a/examples/oauth_client/main.go b/examples/oauth_client/main.go
@@ -129,9 +129,11 @@ func maybeAuthorize(err error) {
 			log.Fatalf("Failed to generate state: %v", err)
 		}
 
-		err = oauthHandler.RegisterClient(context.Background(), "mcp-go-oauth-example")
-		if err != nil {
-			log.Fatalf("Failed to register client: %v", err)
+		if oauthHandler.GetClientID() == "" {
+			err = oauthHandler.RegisterClient(context.Background(), "mcp-go-oauth-example")
+			if err != nil {
+				log.Fatalf("Failed to register client: %v", err)
+			}
 		}
 
 		// Get the authorization URL

Original file line number	Diff line number	Diff line change
`@@ -129,9 +129,11 @@ func maybeAuthorize(err error) {`
`129`	`129`	`log.Fatalf("Failed to generate state: %v", err)`
`130`	`130`	`}`
`131`	`131`
`132`		`- err = oauthHandler.RegisterClient(context.Background(), "mcp-go-oauth-example")`
`133`		`- if err != nil {`
`134`		`- log.Fatalf("Failed to register client: %v", err)`
	`132`	`+ if oauthHandler.GetClientID() == "" {`
	`133`	`+ err = oauthHandler.RegisterClient(context.Background(), "mcp-go-oauth-example")`
	`134`	`+ if err != nil {`
	`135`	`+ log.Fatalf("Failed to register client: %v", err)`
	`136`	`+ }`
`135`	`137`	`}`
`136`	`138`
`137`	`139`	`// Get the authorization URL`