scylladb / alternator-load-balancing

Various tricks, scripts, and libraries, for load balancing multiple Alternator nodes
Apache License 2.0
18 stars 11 forks source link

Feature to override http client #29

Closed dkropachev closed 1 week ago

dkropachev commented 3 months ago

This feature could be used for multiple purposes: from custom retry to ssl CA injection

dkropachev commented 1 week ago

@sylwiaszunejko , can you please review it.

nyh commented 1 week ago

This feature could be used for multiple purposes: from custom retry to ssl CA injection

Please in commit messages the language and if relevant driver version (e.g, Java SDK v1 vs v2) that this commit pertains to. This patch, for example, is for the Go language.

nyh commented 1 week ago

By the way, @dkropachev I see you are writing many "dk/*" branches to this repository. Please consider adding to this public repository only branches that you consider to be useful for everyone (e.g., releases, etc.) - you'd better put your private branches in your own fork of the repository and not push all the branches into the main repsitory. This will make the branch list cleaner for everyone.