aws / aws/aws-sdk-cpp

OverrideOptionsOnConnectionHandle must provide HttpRequest as callback data

Aperta
#1,978 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub
feature-request p2
Lingua principale
C++
Stelle
2.2k
Fork
1.2k
Merge medio
4g 11h
PR unite (30g)
13

Descrizione

### 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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia individuando la dichiarazione e gli utilizzi di OverrideOptionsOnConnectionHandle, quindi esamina CurlHttpClient::MakeRequest e il modo in cui viene invocato il suo gestore della connessione. Conferma come viene rappresentato HttpRequest in quel punto e traccia il percorso del callback. Il lavoro è completato quando il callback riceve l'HttpRequest gestito come dati aggiuntivi, preservando il comportamento previsto del client HTTP personalizzato e tenendo conto della breaking change indicata.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
networking
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.