Making lib/backend/client.getFactory public

Mainly to allow writing "wrapper" custom backends, see eg #278 (comment)
uber · Oct 18, 2020 · bc73e63 · bc73e63
1 parent e435b83
commit bc73e63
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/lib/backend/client.go b/lib/backend/client.go
@@ -32,8 +32,9 @@ func Register(name string, factory ClientFactory) {
  _factories[name] = factory
 }
 
-// getFactory returns backend client factory given client name.
-func getFactory(name string) (ClientFactory, error) {
+// GetFactory returns backend client factory given client name.
+// This function should stay public to allow for wrapper custom backends.
+func GetFactory(name string) (ClientFactory, error) {
  factory, ok := _factories[name]
  if !ok {
  return nil, fmt.Errorf("no backend client defined with name %s", name)

diff --git a/lib/backend/manager.go b/lib/backend/manager.go
@@ -62,7 +62,7 @@ func NewManager(configs []Config, auth AuthConfig) (*Manager, error) {
  var backendConfig interface{}
  for name, backendConfig = range config.Backend { // Pull the only key/value out of map
  }
- factory, err := getFactory(name)
+ factory, err := GetFactory(name)
  if err != nil {
  return nil, fmt.Errorf("get backend client factory: %s", err)
  }