cacert
/
cacert-webdb
8
0
Fork 0
Code Issues Pull Requests 2 Projects Releases Wiki Activity

Labels Milestones

Add migration script for missing users DEFAULTs #4

Merged
jandd merged 1 commits from bugfix/1549-add-missing-defaults-on-users-table into main 10 months ago
Conversation 1 Commits 1 Files Changed 1
jandd commented 10 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 10 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 10 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 10 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 10 months ago
jandd deleted branch bugfix/1549-add-missing-defaults-on-users-table 10 months ago

Reviewers

dirk approved these changes 10 months ago
The pull request has been merged as c9ab54b30a.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
dirk
Labels
Apply labels
Clear labels
  
bug

Something is not working   
duplicate

This issue or pull request already exists   
enhancement

New feature   
help wanted

Need some help   
invalid

Something is wrong   
question

More information is needed   
wontfix

This won't be fixed
No Label
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
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
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'bugfix/1549-add-missing-defaults-on-users-table'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes