OverrideOptionsOnConnectionHandle must provide HttpRequest as callback data
- Vorherrschende Sprache
- C++
- Sterne
- 2.2k
- Forks
- 1.2k
- Ø Merge
- 4 T. 11 Std.
- Gemergte PRs (30 T.)
- 13
Beschreibung
### 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
Beitragsleitfaden
Rechercherichtung
Beginne damit, die Deklaration und die Verwendungen von OverrideOptionsOnConnectionHandle zu finden, und untersuche dann CurlHttpClient::MakeRequest sowie, wie dessen Verbindungs-Handler aufgerufen wird. Bestätige, wie HttpRequest dort dargestellt wird, und verfolge den Callback-Pfad. Die Aufgabe ist abgeschlossen, wenn der Callback den bedienten HttpRequest als zusätzliche Daten erhält, während das beabsichtigte Verhalten des benutzerdefinierten HTTP-Clients erhalten bleibt und die erwähnte Breaking Change berücksichtigt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- networking
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100