Paths

Table of Contentst

Differential D4832

[keyserver] Introduce assertValidDBType
ClosedPublic
Actions

Authored by ashoat on Aug 15 2022, 12:37 PM.

Tags

None

Referenced Files

	F2191153: D4832.id15620.diff
	Thu, Jul 4, 2:12 PM

	Unknown Object (File)
	Tue, Jul 2, 8:55 AM

	Unknown Object (File)
	Tue, Jul 2, 7:46 AM

	Unknown Object (File)
	Mon, Jul 1, 6:41 PM

	Unknown Object (File)
	Mon, Jul 1, 12:14 AM

	Unknown Object (File)
	Wed, Jun 26, 12:55 AM

	Unknown Object (File)
	Wed, Jun 26, 12:54 AM

	Unknown Object (File)
	Wed, Jun 19, 10:48 PM

View All 77 Files

Subscribers

Details

Reviewers

tomek
atul

Commits

rCOMM3b928cdd35d8: [keyserver] Introduce assertValidDBType

Summary

This diff is a simple refactor, and should be a no-op. In the next diff I'll modify assertValidDBType to deprecate MySQL.

Test Plan

Flow
I ran keyserver locally and made sure things generally still worked

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

ashoat created this revision.Aug 15 2022, 12:37 PM

Herald added subscribers: • abosh, • adrian. · View Herald TranscriptAug 15 2022, 12:37 PM

ashoat added a child revision: D4833: [keyserver] Deprecate MySQL support.Aug 15 2022, 12:38 PM

Harbormaster completed remote builds in B11355: Diff 15620.Aug 15 2022, 12:47 PM

ashoat requested review of this revision.Aug 15 2022, 12:47 PM

tomek accepted this revision.Aug 16 2022, 5:34 AM

This revision is now accepted and ready to land.Aug 16 2022, 5:34 AM

(Technically this isn't a pure refactor... if anybody had a bad DBType (neither 'mariadb10.8' nor 'mysql5.7') in their DB config previously, it previously would've defaulted to 'mysql5.7', but now will error. I think this is a good thing.)

Closed by commit rCOMM3b928cdd35d8: [keyserver] Introduce assertValidDBType (authored by ashoat). · Explain WhyAug 16 2022, 6:14 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM3b928cdd35d8: [keyserver] Introduce assertValidDBType.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

database/

19 lines

Diff 15652

keyserver/src/database/db-config.js

Loading...