Add support for custom scheme redirect URIs
Los mantenedores suelen responder en 7 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 65/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- swift
- Área
- authentication
Línea de trabajo
Start at validateRedirectURI in the OAuth flow and trace its callers and related URL-scheme definitions. Check existing validation coverage, then define completion as allowing the requested macOS and iOS custom-scheme callback URLs while preserving the current fragment and loopback-HTTP checks; verify the behavior with the relevant OAuth tests.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Is your feature request related to a problem? Please describe.
When using the OAuth flow in this package for a macOS desktop application, I can only redirect from the authorization flow to http or https URLs.
Describe the solution you'd like
On macOS and iOS it's natural to be able to redirect to apps via custom schemes they claim on the operating system. ASWebAuthenticationSession supports these directly: https://developer.apple.com/documentation/authenticationservices/aswebauthenticationsession/init(url:callback:completionhandler:)-6nut7
ASWebAuthenticationSession ensures that only the calling app’s session receives the authentication callback, even when more than one app registers the same callback URL scheme.
Additional context
Currently, only http and https schemes are supported:
public func validateRedirectURI(_ url: URL) throws {
guard let components = URLComponents(url: url, resolvingAgainstBaseURL: false),
let scheme = components.scheme?.lowercased(),
components.fragment == nil
else {
throw OAuthAuthorizationError.invalidRedirectURI(url.absoluteString)
}
if scheme == OAuthURLScheme.https { return }
if scheme == OAuthURLScheme.http,
let host = components.host?.lowercased(),
OAuthLoopbackHost.isLoopback(host)
{
return
}
throw OAuthAuthorizationError.invalidRedirectURI(url.absoluteString)
}
- Lenguaje dominante
- Swift
- Estrellas
- 1.5k
- Forks
- 254
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de modelcontextprotocol/swift-sdk
-
NetworkTransport leaks the underlying socket (CLOSE_WAIT) when reconnection is disabled and the receive loop terminatesPosiblemente ocupada @skirrellyjones la tomó hace 43 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
modelcontextprotocol/swift-sdk#282 ·
Los mantenedores suelen responder en 7 días
-
Value decoding changes data-URL-looking JSON strings into Value.dataPosiblemente ocupada @bitbemol la tomó hace 54 días. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
modelcontextprotocol/swift-sdk#277 ·
Los mantenedores suelen responder en 7 días
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
modelcontextprotocol/swift-sdk#274 ·
Los mantenedores suelen responder en 7 días
-
`HTTPClientTransport` fails to build on Windows: `#if !os(Linux)` guards no longer match the `EventSource` platform conditionQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
modelcontextprotocol/swift-sdk#261 ·
Los mantenedores suelen responder en 7 días
-
Value: add numberValue (Double?) that coerces .int and .doublePosiblemente ocupada @gsdali la tomó hace 165 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
modelcontextprotocol/swift-sdk#225 ·
Los mantenedores suelen responder en 7 días
Todos los issues de modelcontextprotocol/swift-sdk
Issues similares
-
a:false-positive
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día
-
Idle CPU stays at ~8% after the "Text wasn't inserted" card is shown oncePosiblemente ocupada @nickbecker la tomó hoy. Abiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
altic-dev/FluidVoice#1217 ·
Los mantenedores suelen responder en 1 día
-
BrainBar Watcher chart: unavailable-flow tooltip/accessibility drops the health reason for .attention/.stoppedPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
EtanHey/brainlayer#1439 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
alejandrobujan/tendedero#98 ·
Los mantenedores suelen responder en 2 días
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
vorssaint/vorssaint-utils#3368 ·
Los mantenedores suelen responder en 1 día