[keyserver] Type ConnectionOptions passed to mysql2 library
ClosedPublic
Actions

Authored by ashoat on Jul 13 2022, 4:50 PM.

Details

Reviewers

• jon
atul
tomek

Commits

rCOMMd402dc57f2e8: [keyserver] Type ConnectionOptions passed to mysql2 library

Summary

We should type this. (We use types from the mysql library because mysql2 is meant to be a drop-in replacement, and there is no Flow libdef for mysql2.)

Depends on D4515

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Jul 13 2022, 4:50 PM

Herald added subscribers: • abosh, • adrian. · View Herald TranscriptJul 13 2022, 4:50 PM

ashoat added a child revision: D4516: [keyserver] Add dbType to DBConfig.Jul 13 2022, 4:57 PM

Make PoolOptions and ConnectionOptions exact objects to be consistent with this runtime warning we get from mysql2:

Ignoring invalid configuration option passed to Connection: dbType. This is currently a warning, but in future versions of MySQL2, an error will be thrown if you pass an invalid configuration option to a Connection

Harbormaster completed remote builds in B10501: Diff 14449.Jul 13 2022, 5:02 PM

Harbormaster completed remote builds in B10503: Diff 14451.Jul 13 2022, 5:11 PM

ashoat requested review of this revision.Jul 13 2022, 5:11 PM

atul accepted this revision.Jul 14 2022, 9:54 AM

This revision is now accepted and ready to land.Jul 14 2022, 9:54 AM

Closed by commit rCOMMd402dc57f2e8: [keyserver] Type ConnectionOptions passed to mysql2 library (authored by ashoat). · Explain WhyJul 17 2022, 8:13 PM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMd402dc57f2e8: [keyserver] Type ConnectionOptions passed to mysql2 library.

ashoat mentioned this in D4676: [keyserver] Avoid passing dbType to mysql2 in getMigrationConnection.Jul 28 2022, 11:21 PM

ashoat mentioned this in rCOMMa89052e0558c: [keyserver] Avoid passing dbType to mysql2 in getMigrationConnection.Jul 29 2022, 8:33 AM