aws / aws/amazon-q-developer-cli

Use purposeful api from rmcp to override default client name

Offen
#3,229 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
Rust
Sterne
2k
Forks
439
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Since the release of RMCP 0.8.0, they have added a way to change the client name used for oauth.
https://github.com/modelcontextprotocol/rust-sdk/pull/469/files

Currently, this was achieved using a somewhat convoluted hack:
https://github.com/aws/amazon-q-developer-cli/blob/1fa1d74976e119459e99d50ccc40c87195de5547/crates/chat-cli/src/mcp_client/oauth_util.rs#L566-L621

We can get rid of this hack and just use the api as provided by the RMCP crate.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in crates/chat-cli/src/mcp_client/oauth_util.rs etwa in den Zeilen 566–621 und vergleiche die in PR 469 beschriebene RMCP 0.8.0 API. Ersetze die umständliche Client-Namensüberschreibung durch die bereitgestellte API und überprüfe anschließend, dass der OAuth-Client weiterhin den vorgesehenen Namen verwendet.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
rust
Bereich
authentication, cli
Issue-Typ
Refactoring
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.