x/website: MySQL config is incorrect in the database access tutorial.

[kohkimakimoto]

MySQL config code should use AllowNativePasswords: true.
The tutorial https://golang.org/doc/tutorial/database-access describes MySQL config code as the following:

cfg := mysql.Config{
	User:   os.Getenv("DBUSER"),
	Passwd: os.Getenv("DBPASS"),
	Net:    "tcp",
	Addr:   "127.0.0.1:3306",
	DBName: "recordings",
}

But, it causes an error like the following:

[mysql] 2021/10/24 17:03:01 connector.go:95: could not use requested auth plugin 'mysql_native_password': this user requires mysql native password authentication.
2021/10/24 17:03:01 this user requires mysql native password authentication.

To run the code correctly, we should add AllowNativePasswords: true to the MySQL config or use msql.NewConfig method instead of creating mysql.Config struct directly. Please see the following full config code.

The correct code is:

cfg := mysql.Config{
	User:   os.Getenv("DBUSER"),
	Passwd: os.Getenv("DBPASS"),
	Net:    "tcp",
	Addr:   "127.0.0.1:3306",
	DBName: "recordings",
	AllowNativePasswords: true,
}

or

cfg := mysql.NewConfig()
cfg.User = os.Getenv("DBUSER")
cfg.Passwd = os.Getenv("DBPASS")
cfg.Net = "tcp"
cfg.Addr = "127.0.0.1:3306"
cfg.DBName = "recordings"

See also:

• Mark AllowNativePassword in DSN param string be true by default. go-sql-driver/mysql#644
• https://github.com/golang/website/blob/281050ffddd88206d068f5f9e9cfa31cf01c9446/_content/doc/tutorial/database-access.md?plain=1#L243-L249
• https://github.com/golang/website/blob/281050ffddd88206d068f5f9e9cfa31cf01c9446/_content/doc/tutorial/database-access.md?plain=1#L692-L698

[seankhliao]

maybe it should use mysql.NewConfig() to set defaults.

Also for doc/database/open-handle

cc @stevetraut

[gopherbot]

Change https://golang.org/cl/358294 mentions this issue: _content/doc: update mysql.Config to mysql.NewConfig

[thomas-cools]

This is still an issue unless I'm looking at an outdated version of this page: https://go.dev/doc/tutorial/database-access... I don't see the change to reference mysql.NewConfig, instead I still see

 cfg := mysql.Config{
        User:   os.Getenv("DBUSER"),
        Passwd: os.Getenv("DBPASS"),
        Net:    "tcp",
        Addr:   "127.0.0.1:3306",
        DBName: "recordings",
    }

[sprive]

The Go MySQL tutorial fails without: AllowNativePasswords: true,
https://go.dev/doc/tutorial/database-access

@dr2chase or @eliben - Could you sponsor this doc change, or review with team?
https://github.com/golang/website/pull/242/checks

Please note during the 2 year window this issue is open, Go Twitter promoted it: https://twitter.com/golang/status/1641133293180420116

[eliben]

Thanks for bringing this to our attention, @sprive

I note that an earlier CL addressing this already exists in Gerrit: https://go-review.googlesource.com/c/website/+/358294 (it's mentioned here upthread). I will take a look at the CL and leave some comments - please help review it and update it, if needed.