Fix auth being not passed in service config#33
Fix auth being not passed in service config#33AzeemMuzammil merged 5 commits intoballerina-platform:mainfrom
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR fixes authentication configuration by properly passing auth settings in service configurations and updating MCP library version to 1.0.2. The changes ensure auth configuration is correctly applied across all clients and servers.
- Updates MCP library dependency from version 1.0.1 to 1.0.2 across all example projects
- Fixes auth configuration not being passed in HTTP client and service configurations
- Updates certificate and private key files with corrected paths and new certificates
Reviewed Changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| ballerina/streamable_http.bal | Refactors HTTP response handling logic and improves error status code validation |
| ballerina/dispatcher_service.bal | Adds authentication calls to HTTP resource methods |
| examples/clients/mcp-shopping-client/main.bal | Adds comprehensive JWT authentication configuration for HTTP client |
| examples/servers/mcp-shopping-server/main.bal | Updates JWT configuration with corrected certificate path and audience |
| Various .toml files | Updates MCP library version to 1.0.2 and dependency versions |
| Certificate/key files | Replaces old certificates with new ones and corrects file paths |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #33 +/- ##
============================================
- Coverage 17.60% 17.49% -0.11%
Complexity 3 3
============================================
Files 15 15
Lines 642 646 +4
Branches 170 171 +1
============================================
Hits 113 113
- Misses 528 532 +4
Partials 1 1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
MohamedSabthar
left a comment
There was a problem hiding this comment.
@AzeemMuzammil Let's link the issues. Changes LGTM.
Purpose
$title.
Fixes: ballerina-platform/ballerina-library#8419
Examples
Checklist