aws / aws/aws-sdk-cpp

OverrideOptionsOnConnectionHandle must provide HttpRequest as callback data

Ouverte
#1,978 2 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
feature-request p2
Langage dominant
C++
Étoiles
2.2k
Forks
1.2k
Merge moyen
4 j 11 h
PR mergées (30 j)
13

Description

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

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par localiser la déclaration et les utilisations de OverrideOptionsOnConnectionHandle, puis examinez CurlHttpClient::MakeRequest et la manière dont son gestionnaire de connexion est invoqué. Confirmez comment HttpRequest est représenté à cet endroit et suivez le chemin du callback. La tâche est terminée lorsque le callback reçoit le HttpRequest traité en tant que données supplémentaires, tout en préservant le comportement prévu du client HTTP personnalisé et en tenant compte du breaking change indiqué.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
networking
Type d'issue
Fonctionnalité
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.