OverrideOptionsOnConnectionHandle must provide HttpRequest as callback data
- Lenguaje dominante
- C++
- Estrellas
- 2.2k
- Forks
- 1.2k
- Merge medio
- 4 d 11 h
- PR fusionados (30 d)
- 13
Descripción
### Describe the feature
Adding custom SetHttpClientFactory and custom CurlHttpClient if great but there is no link between client and served request in function OverrideOptionsOnConnectionHandle.
### Use Case
Implementing simultaneous and dynamic SSL checking and verification via SSL_CTX_set_verify (check for some connections, discard for other connections, etc.) with CURLOPT_SSL_CTX_FUNCTION is not possible because HttpClientFactory is static and there is no way to understand which option to use in OverrideOptionsOnConnectionHandle.
### Proposed Solution
Declare OverrideOptionsOnConnectionHandle with additional parameter HttpRequest* and when invoked in CurlHttpClient::MakeRequest pass "request.get()" along with curl connection handler.
### Other Information
_No response_
### Acknowledgements
- [ ] I may be able to implement this feature request
- [X] This feature might incur a breaking change
Guía de contribución
Línea de trabajo
Comienza localizando la declaración y los usos de OverrideOptionsOnConnectionHandle; después, inspecciona CurlHttpClient::MakeRequest y cómo se invoca su manejador de conexión. Confirma cómo se representa HttpRequest allí y sigue la ruta del callback. El trabajo estará terminado cuando el callback reciba el HttpRequest atendido como datos adicionales, preservando el comportamiento previsto del cliente HTTP personalizado y teniendo en cuenta el breaking change mencionado.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- networking
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100