phiilu / mailman

Mailman is a GUI to help you manage your email accounts stored in a MySQL/MariaDB database.
MIT License
70 stars 10 forks source link

Problem with Database #37

Closed Rudi1986 closed 5 years ago

Rudi1986 commented 5 years ago

Hi,

i just made an update and after it Mailman won't let me log in anymore. mailman-error-1.log says: "Error: No database selected". After playing around with .env (new Variable) i took a closer lock at your last changes in db.js. Especially the line where the DB is selected. Old: db: process.env.MAILMAN_DB_DATABASE New: database: process.env.MAILMAN_DB_DATABASE

I changed this line back to the old version. And it works just fine.

Greetings from Munich Rudi

hibas123 commented 5 years ago

Did you run npm install after the update?

Rudi1986 commented 5 years ago

Yes.

phiilu commented 5 years ago

You might be right, that this will cause a problem. I merged this PR and indeed this line was changed.

I think in a newer version of knex (the database client) the field is called database instead of db.

I am working on a newer version of Mailman and hope I can release it till the end of this year. Not so easy to find time anymore.

Rudi1986 commented 5 years ago

No Problem... Took about 30 minutes to get it up again but works fine now. Just posted this for your information and if someone runs into the same issue.