cashapp / blip Goto Github PK

Blip appears to have a bug with reloading credentials if the port was not explicitly set when configuring a monitor. When the credential reload function for a hostname is registered it uses the hostname provided in the configuration:

DEBUG reload_password.go:38 added myhostname

When the reload callback is run it loads the hostname from the DSN, but the driver explicitly adds the default port if none is specified. This results in an error because the passed hostname does not match the one that was registered:

DEBUG reload_password.go:58 no credential func for hostname:3306

The workaround for now is to explicitly include the default port when setting the hostname on the monitor configuration.