modelcontextprotocol/rust-sdk
Watch
1
0
Fork
You've already forked rust-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: do not manually construct fallback authorization metadata (#507)

Browse source
This commit is contained in:
Alex Hancock 2025-11-03 12:08:32 -05:00 • committed by GitHub
commit d3ddc09e52
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 3 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

26
crates/rmcp/src/transport/auth.rs
View file

@ -251,29 +251,9 @@ impl AuthorizationManager {
return Ok(metadata); return Ok(metadata);
} }
warn!("No valid authorization metadata found, falling back to default endpoints"); // No valid authorization metadata found - return error instead of guessing
// OAuth endpoints must be discovered from the server, not constructed by the client
// fallback to default endpoints Err(AuthError::NoAuthorizationSupport)
let mut auth_base = self.base_url.clone();
// discard the path part, only keep scheme, host, port
auth_base.set_path("");
// Helper function to create endpoint URL
let create_endpoint = |path: &str| -> String {
let mut url = auth_base.clone();
url.set_path(path);
url.to_string()
};
Ok(AuthorizationMetadata {
authorization_endpoint: create_endpoint("authorize"),
token_endpoint: create_endpoint("token"),
registration_endpoint: None,
issuer: None,
jwks_uri: None,
scopes_supported: None,
additional_fields: HashMap::new(),
})
} }
/// get client id and credentials /// get client id and credentials