Skip to content

Draft: POC: Use ConnectionHandler to implement transparent connection proxy

Kamil Trzciński requested to merge use-connection-handler into master

What does this MR do?

I was wondering how differently we could enable support for LB in a transparent way to work across any type of connects_to : specification.

This tries to implement a ConnectionHandler which does create a connection proxy for the purpose of LB depending on a configuration.

This appears to work for me transparently to support any number of connection proxies.

The SD should work for main database with these changes.

Ah.: We need to pass actual db_config as well down to LB/ConnectionPool that we know at this moment.

Conformity

Availability and Testing

Security

Does this MR contain changes to processing or storing of credentials or tokens, authorization and authentication methods or other items described in the security review guidelines? If not, then delete this Security section.

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Kamil Trzciński

Merge request reports