Merge pull request brianc#2217 from charmander/normal-encoding-parameter

brianc · web-flow · commit d5b615e98e23 · 2020-05-15T12:54:44.000-05:00
Send the `client_encoding` startup parameter value with more typical formatting
diff --git a/packages/pg-protocol/src/outbound-serializer.test.ts b/packages/pg-protocol/src/outbound-serializer.test.ts
@@ -18,7 +18,7 @@ describe('serializer', () => {
         .addCString('database')
         .addCString('bang')
         .addCString('client_encoding')
-        .addCString("'utf-8'")
+        .addCString('UTF8')
         .addCString('')
         .join(true)
     )
diff --git a/packages/pg-protocol/src/serializer.ts b/packages/pg-protocol/src/serializer.ts
@@ -25,7 +25,7 @@ const startup = (opts: Record<string, string>): Buffer => {
     writer.addCString(key).addCString(opts[key])
   }
 
-  writer.addCString('client_encoding').addCString("'utf-8'")
+  writer.addCString('client_encoding').addCString('UTF8')
 
   var bodyBuffer = writer.addCString('').flush()
   // this message is sent without a code

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,7 @@ const startup = (opts: Record<string, string>): Buffer => {`
`25`	`25`	`writer.addCString(key).addCString(opts[key])`
`26`	`26`	`}`
`27`	`27`
`28`		`- writer.addCString('client_encoding').addCString("'utf-8'")`
	`28`	`+ writer.addCString('client_encoding').addCString('UTF8')`
`29`	`29`
`30`	`30`	`var bodyBuffer = writer.addCString('').flush()`
`31`	`31`	`// this message is sent without a code`