fix(postgraphql): Throw if jwtSecret set but jwtPgTypeIndentifer isn't (

graphile#466)

* fix(postgraphql): Throw an error if jwtSecret is provided without jwtPgTypeIndentifer being provided

* Tweaks

* Remove semicolon

* Adding test

src/postgraphql/__tests__/postgraphql-test.js

@@ -138,3 +138,8 @@ test('will create a new PostGraphQL schema on when `watchPgSchemas` emits a chan
   ])
   console.log = origLog
 })
+
+test('will error if jwtSecret is provided without jwtPgTypeIdentifier', async () => {
+  const pgPool = new Pool()
+  expect(() => postgraphql(pgPool, [], { jwtSecret: 'test' })).toThrow()
+})

src/postgraphql/postgraphql.ts

@@ -64,6 +64,12 @@ export default function postgraphql (
     options = schemaOrOptions
   }
 
+  // Check for a jwtSecret without a jwtPgTypeIdentifier
+  // a secret without a token identifier prevents JWT creation
+  if (options.jwtSecret && !options.jwtPgTypeIdentifier) {
+    throw new Error('jwtSecret provided, however jwtPgTypeIdentifier (token identifier) not provided.')
+  }
+
   // Creates the Postgres schemas array.
   const pgSchemas: Array<string> = Array.isArray(schema) ? schema : [schema]