Added one more GCP connection string example for flexible environment.

[cn007b]

Description

Added one more GCP connection string example for flexible environment.

Checklist

• Code compiles correctly
• Created tests which fail without the change (if possible)
• All tests passing
• Extended the README / documentation, if necessary
• Added myself / the copyright holder to the AUTHORS file

[methane]

Would you confirm an example in Standard 2nd example is OK?

As far as I can see, Google uses unix protocol in their example. (see here)

cloudsql protocol requires // +build appengine build tag. I am not sure this build tag is enabled for Standard 2nd environment.

[cn007b]

From documentation:

standard
sql.Open("mysql", fmt.Sprintf("%s:%s@cloudsql(%s)/", user, password, connectionName))

flexible
MYSQL_CONNECTION: user:password@unix(/cloudsql/INSTANCE_CONNECTION_NAME)/dbname

[methane]

standard
sql.Open("mysql", fmt.Sprintf("%s:%s@cloudsql(%s)/", user, password, connectionName))

It is Go 1.9. It's 1st gen. It will be die soon. (No new deply since 2019-10-01)

Go 1.11 and 1.12 are 2nd gen. It uses unix.
https://cloud.google.com/appengine/docs/standard/go111/using-cloud-sql

[cn007b]

Yes, true, you're right! For 2nd gen:
dbURI := fmt.Sprintf("%s:%s@unix(%s/%s)/%s", user, password, socket, connectionName, dbName).
I'll update standard 2nd gen example.

[methane]

I'll update standard 2nd gen example.

Please unify 2nd gen example and flexible example if they are same.

[cn007b]

Does it look better?

[methane]

LGTM, but travis fail blocks merging this.

[cn007b]

Is it possible to me to do something with travis?

[methane]

No. I already made PR to fix it but I can not merge it. Only another maintainer can help.

This project needs at least two active maintainers to go forward. Sorry.

[julienschmidt]

@cn007b could you please rebase your changes onto the current upstream master?
It contains a fix for the macOS builds now.

[cn007b]

Done.