Merge ~lvoytek/ubuntu/+source/mysql-8.0:lp1980466-fix-mysql_secure_installation-mantic into ubuntu/+source/mysql-8.0:ubuntu/devel

Proposed by Lena Voytek
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merge reported by: Lena Voytek
Merged at revision: c7140ff65d2b540e6f91ebba7effc99ce6a46ea2
Proposed branch: ~lvoytek/ubuntu/+source/mysql-8.0:lp1980466-fix-mysql_secure_installation-mantic
Merge into: ubuntu/+source/mysql-8.0:ubuntu/devel
Diff against target: 60 lines (+38/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/mysql_secure_installation-remove-root-pw-creation.patch (+30/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
git-ubuntu bot Approve
Miriam España Acebal (community) Approve
Canonical Server Reporter Pending
Review via email: mp+442678@code.launchpad.net

Description of the change

Adding mysql_secure_installation fix to mantic

PPA: https://launchpad.net/~lvoytek/+archive/ubuntu/mysql-mysql-secure-installation

Lunar mp (will reupload since security team superseded it): https://code.launchpad.net/~lvoytek/ubuntu/+source/mysql-8.0/+git/mysql-8.0/+merge/440515

To post a comment you must log in.
635018f... by Marc Deslauriers

8.0.33-0ubuntu3 (patches unapplied)

Imported using git-ubuntu import.

Revision history for this message
Miriam España Acebal (mirespace) wrote :

LGTM Lena! Same as Lunar :)

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: lvoytek, mirespace
Uploaders: lvoytek
MP auto-approved

review: Approve
c7140ff... by Lena Voytek

changelog

Revision history for this message
Lena Voytek (lvoytek) wrote :

Thanks for the review Miriam! Updated and uploaded:

dput ubuntu ../mysql-8.0_8.0.33-0ubuntu4_source.changes
D: Setting host argument.
Checking signature on .changes
gpg: ../mysql-8.0_8.0.33-0ubuntu4_source.changes: Valid signature from 34B8AD7D9529E793
Checking signature on .dsc
gpg: ../mysql-8.0_8.0.33-0ubuntu4.dsc: Valid signature from 34B8AD7D9529E793
Uploading to ubuntu (via ftp to upload.ubuntu.com):
  Uploading mysql-8.0_8.0.33-0ubuntu4.dsc: done.
  Uploading mysql-8.0_8.0.33-0ubuntu4.debian.tar.xz: done.
  Uploading mysql-8.0_8.0.33-0ubuntu4_source.buildinfo: done.
  Uploading mysql-8.0_8.0.33-0ubuntu4_source.changes: done.
Successfully uploaded packages.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index a1641cf..e5a5324 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+mysql-8.0 (8.0.33-0ubuntu4) mantic; urgency=medium
7+
8+ * d/p/mysql_secure_installation-remove-root-pw-creation.patch: Fix
9+ mysql_secure_installation by removing root password creation (LP: #1980466)
10+
11+ -- Lena Voytek <lena.voytek@canonical.com> Mon, 15 May 2023 08:52:09 -0700
12+
13 mysql-8.0 (8.0.33-0ubuntu3) mantic; urgency=medium
14
15 * Fix crash on startup on armhf (LP: #2019203)
16diff --git a/debian/patches/mysql_secure_installation-remove-root-pw-creation.patch b/debian/patches/mysql_secure_installation-remove-root-pw-creation.patch
17new file mode 100644
18index 0000000..22bc248
19--- /dev/null
20+++ b/debian/patches/mysql_secure_installation-remove-root-pw-creation.patch
21@@ -0,0 +1,30 @@
22+Description: Remove the creation of a root password in mysql_secure_installation
23+ Since Ubuntu and Debian use the auth_socket plugin for the root user by default,
24+ and attempting to set a password for it through "SET_PASSWORD" causes an error,
25+ do not create a root password in the mysql_secure_installation script if one
26+ has not been set previously. If the user would like to instead use a password
27+ they can run
28+ ALTER USER 'root'@'localhost' IDENTIFIED WITH mysql_native_password by 'password';
29+ in the mysql client.
30+Author: Lena Voytek <lena.voytek@canonical.com>
31+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/mysql-8.0/+bug/1980466
32+Forwarded: no
33+Last-Update: 2023-04-20
34+---
35+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
36+--- a/client/mysql_secure_installation.cc
37++++ b/client/mysql_secure_installation.cc
38+@@ -811,8 +811,11 @@
39+ }
40+
41+ if (!hadpass) {
42+- fprintf(stdout, "Please set the password for %s here.\n", opt_user);
43+- set_opt_user_password(component_set);
44++ fprintf(stdout,
45++ "\nSkipping password set for %s as authentication with auth_socket is used by default.\n"
46++ "If you would like to use password authentication instead, this can be done with the \"ALTER_USER\" command.\n"
47++ "See https://dev.mysql.com/doc/refman/8.0/en/alter-user.html#alter-user-password-management for more information.\n\n"
48++ , opt_user);
49+ } else if (opt_use_default == false) {
50+ char prompt[256];
51+ fprintf(stdout, "Using existing password for %s.\n", opt_user);
52diff --git a/debian/patches/series b/debian/patches/series
53index f36ae9f..dab82cf 100644
54--- a/debian/patches/series
55+++ b/debian/patches/series
56@@ -9,3 +9,4 @@ boost-1.76.0-fix_multiprecision_issue_419-ppc64le.patch
57 disable_timestamping_test.patch
58 revert-be8348a7.patch
59 fix_expired_date_in_test.patch
60+mysql_secure_installation-remove-root-pw-creation.patch

Subscribers

People subscribed via source and target branches