Is it possible to automatically generate keypair using opendkim-genkey?
Also, IMO, would be nice to show only public key as DNS record, to just copy-paste it

Is it possible to automatically generate keypair using opendkim-genkey? Also, IMO, would be nice to show only public key as DNS record, to just copy-paste it

Probably possible, though I don't have the time to add it in this PR. Also wanted to keep it relatively simple to start of with.

Is it possible to automatically generate keypair using opendkim-genkey? Also, IMO, would be nice to show only public key as DNS record, to just copy-paste it

Yes, I've been thinking the same :)

I've only not merged it because I've not had time to try using it ....

Code works on PostgreSQL now - the original SQL in upgrade.php didn't work with PostgreSQL (it didn't like backticks, it didn't like an index being created in the table spec) - so I just copied it into it's own _pgsql type function.

I did two follow-up fixes:

• 6a53b1a: Drop duplicate db_delete() call from DkimHandler and DkimsigningHandler. The hardcoded name is the same as $this->db_table, which effectively means running the same query twice. One of them is enough.
• a8b4e3f: Update translations with DKIM texts so that DKIM support also works in other languages (done with language-update.sh --patch)

I also have a question: Would it be ok for you to rename the txtlarge field type to txta (short for "textarea")? That would have the small advantage to keep the type "column" in init_struct() limited to 4 chars, instead of widening it just for one field type.

I did two follow-up fixes:

* [6a53b1a](https://github.com/postfixadmin/postfixadmin/commit/6a53b1ab882446dda7111d63664afb06f4adf9af): Drop duplicate db_delete() call from DkimHandler and DkimsigningHandler. The hardcoded name is the same as $this->db_table, which effectively means running the same query twice. One of them is enough.

* [a8b4e3f](https://github.com/postfixadmin/postfixadmin/commit/a8b4e3f4358425ef06b08397637b31ac25e00be5): Update translations with DKIM texts so that DKIM support also works in other languages (done with `language-update.sh --patch`)

I also have a question: Would it be ok for you to rename the txtlarge field type to txta (short for "textarea")? That would have the small advantage to keep the type "column" in init_struct() limited to 4 chars, instead of widening it just for one field type.

I see no problem with that

txtlarge -> txta rename done.

Is it possible to automatically generate keypair using opendkim-genkey? Also, IMO, would be nice to show only public key as DNS record, to just copy-paste it

Yes, I've been thinking the same :)

I've only not merged it because I've not had time to try using it ....

@DavidGoodwin This would be a great feature to add :-)