search

yogthos / migratus

MIGRATE ALL THE THINGS!
641 stars 95 forks source link

When error happens, db password information is shown #220

Closed pinkfrog9 closed 1 year ago

pinkfrog9 commented 1 year ago

This piece of information should better be hidden.

yogthos commented 1 year ago

Agreed, this would be good to omit in the log. Would you be up for making a PR for this by any chance?

pinkfrog9 commented 1 year ago

Hi @yogthos. Would you please look at: https://github.com/yogthos/migratus/pull/222

yogthos commented 1 year ago

Thanks looks good, there are a few other fixes in the works, so I'll cut a new release next week.

ieugen commented 1 year ago

Relates to https://github.com/yogthos/migratus/issues/189 . Possible duplicate.

ieugen commented 1 year ago

This was released as part of 1.4.0. Thanks @pinkfrog9 !