microsoft / microsoft/libHttpClient
There should be a version of HCHttpCallRequestSetSSLValidation for web sockets
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C++
- Sterne
- 295
- Forks
- 144
- Ø Merge
- 2 T. 19 Std.
- Gemergte PRs (30 T.)
- 2
Beschreibung
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.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die vorhandenen Einstiegspunkte HCHttpCallRequestSetSSLValidation und HCWebSocketSetProxyDecryptsHttps zu lesen, um zu verstehen, wie die SSL-Validierung für HTTP- und WebSocket-Verbindungen gesteuert wird. Füge anschließend die entsprechende WebSocket-seitige API hinzu, die im Issue beschrieben ist, und überprüfe, dass sich ein lokaler WebSocket-Echo-Server mit einem selbstsignierten Zertifikat verbinden kann, wenn die Validierung deaktiviert ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- api, security
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 54/100