Kienan Stewart authored Jan 12, 2023 and chris committed Jan 27, 2023

The return value passed to m_action::finish() is inserted into the
action table's status column. If an action doesn't have a specific
return set by the output of the job it ran, then 'OK' was being passed
through to the database. Consequently the query in m_action::finish()
always failed.

misc: prevent a few undefined index / variable / trying to access array offset on value of type bool

 this prevents ERROR 1364 (HY000): Field 'https' doesn't have a default value
 when adding dkim/spf/dmarc records

This reverts commit ad5fcd5b.