microsoft / microsoft/libHttpClient
There should be a version of HCHttpCallRequestSetSSLValidation for web sockets
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C++
- Estrellas
- 295
- Forks
- 144
- Merge medio
- 2 d 19 h
- PR fusionados (30 d)
- 2
Descripción
I am adding support for Web Sockets to our title, and have been testing my work with a very simple locally run echo server. Everything works fine when testing over http, but switching to https fails as I am just using a self signed certificate for local testing.
For making HTTP calls, I call HCHttpCallRequestSetSSLValidation to skip the ssl validation, but there is no equivilent for web sockets. I found HCWebSocketSetProxyDecryptsHttps, and using a debugger to skip the empty proxy check was able to get a connection open over https with the ssl check bypassed. This is not sustainable in the long term, as I can't expect other developers to stick a breakpoint in a random bit of code and set a random variable to true.
Please consider adding a new function along the lines of HCWebSocketSetSSLValidation, which when called disables the SSL validation step.
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.
Línea de trabajo
Comienza leyendo los puntos de entrada existentes HCHttpCallRequestSetSSLValidation y HCWebSocketSetProxyDecryptsHttps para entender cómo se controla la validación SSL para las conexiones HTTP y WebSocket. Añade la API correspondiente orientada a WebSocket descrita en el issue y, después, verifica que un servidor WebSocket de eco local que use un certificado autofirmado pueda conectarse cuando la validación esté deshabilitada.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- api, security
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 54/100