Add migration script for missing users DEFAULTs #4

Merged
jandd merged 1 commits from bugfix/1549-add-missing-defaults-on-users-table into main 7 months ago
jandd commented 7 months ago
Owner

This commit adds a migration script to add missing DEFAULT values on the
users table. INSERTs into the users table fail without these DEFAULTs on
MariaDB in strict mode.

See https://bugs.cacert.org/view.php?id=1549

This commit adds a migration script to add missing DEFAULT values on the users table. INSERTs into the users table fail without these DEFAULTs on MariaDB in strict mode. See https://bugs.cacert.org/view.php?id=1549
jandd added 1 commit 7 months ago
10540621a8 Add migration script for missing users DEFAULTs
This commit adds a migration script to add missing DEFAULT values on the
users table. INSERTs into the users table fail without these DEFAULTs on
MariaDB in strict mode.
ted commented 7 months ago

Just in case this is relevant, I have reviewed the changes, and they are a PASS insofar that there is no harmful content.

I have not tried to verify that the change fixes the problem.

Just in case this is relevant, I have reviewed the changes, and they are a PASS insofar that there is no harmful content. I have not tried to verify that the change fixes the problem.
dirk approved these changes 7 months ago
dirk left a comment
Owner

review passed from my site (as well as from ted regarding his comment to this patch request)

review passed from my site (as well as from ted regarding his comment to this patch request)
jandd merged commit c9ab54b30a into main 7 months ago
jandd deleted branch bugfix/1549-add-missing-defaults-on-users-table 7 months ago

Reviewers

dirk approved these changes 7 months ago
The pull request has been merged as c9ab54b30a.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cacert/cacert-webdb#4
Loading…
There is no content yet.