@opensaas/keystone-nextjs-auth 20.1.0 → 20.3.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # @opensaas-keystone/nextjs-auth
 
+## 20.3.0
+
+### Minor Changes
+
+- 34e9932: Fix up stale next session - session is now refreshed on change of data
+
+## 20.2.0
+
+### Minor Changes
+
+- 844f069: Generate NodeAPI by def, minor cleanups
+
+### Patch Changes
+
+- 6d63b1f: Minor Patch upgrades
+
+## 20.1.1
+
+### Patch Changes
+
+- 0b22f90: Fix Error where signin was returning deny on create user
+
 ## 20.1.0
 
 ### Minor Changes

package/dist/declarations/src/lib/validateNextAuth.d.ts

@@ -1,6 +1,6 @@
 import type { KeystoneListsAPI } from '@keystone-6/core/types';
 import { NextAuthErrorCode } from '../types';
-export declare function validateNextAuth(list: any, identityField: string, identity: string | number, protectIdentities: boolean, itemAPI: KeystoneListsAPI<any>[string]): Promise<{
+export declare function validateNextAuth(identityField: string, identity: string | number, protectIdentities: boolean, itemAPI: KeystoneListsAPI<any>[string]): Promise<{
     success: false;
     code: NextAuthErrorCode;
 } | {

package/dist/declarations/src/types.d.ts

@@ -13,7 +13,7 @@ export declare type NextAuthSession = {
     itemId: string;
     data: any;
 };
-export declare type NextAuthProviders = [Provider];
+export declare type NextAuthProviders = Provider[];
 declare type KeytoneAuthProviders = {
     providers: NextAuthProviders;
 };

package/dist/opensaas-keystone-nextjs-auth.cjs.dev.js

@@ -490,6 +490,9 @@ function createAuth({
       session,
       providers,
       lists: _objectSpread({}, keystoneConfig.lists),
+      experimental: _objectSpread(_objectSpread({}, keystoneConfig.experimental), {}, {
+        generateNodeAPI: true
+      }),
       extendGraphqlSchema: existingExtendGraphQLSchema ? schema => existingExtendGraphQLSchema(extendGraphqlSchema(schema)) : extendGraphqlSchema
     });
   };

package/dist/opensaas-keystone-nextjs-auth.cjs.prod.js

@@ -486,6 +486,9 @@ function createAuth({
       session,
       providers,
       lists: _objectSpread({}, keystoneConfig.lists),
+      experimental: _objectSpread(_objectSpread({}, keystoneConfig.experimental), {}, {
+        generateNodeAPI: true
+      }),
       extendGraphqlSchema: existingExtendGraphQLSchema ? schema => existingExtendGraphQLSchema(extendGraphqlSchema(schema)) : extendGraphqlSchema
     });
   };

package/dist/opensaas-keystone-nextjs-auth.esm.js

@@ -457,6 +457,9 @@ function createAuth({
       session,
       providers,
       lists: _objectSpread({}, keystoneConfig.lists),
+      experimental: _objectSpread(_objectSpread({}, keystoneConfig.experimental), {}, {
+        generateNodeAPI: true
+      }),
       extendGraphqlSchema: existingExtendGraphQLSchema ? schema => existingExtendGraphQLSchema(extendGraphqlSchema(schema)) : extendGraphqlSchema
     });
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opensaas/keystone-nextjs-auth",
-  "version": "20.1.0",
+  "version": "20.3.0",
   "repository": "https://github.com/opensaasau/keystone-nextjs-auth",
   "license": "MIT",
   "main": "dist/opensaas-keystone-nextjs-auth.cjs.js",
@@ -10,11 +10,11 @@
     "@babel/runtime-corejs3": "^7.16.8",
     "@types/ejs": "^3.1.0",
     "cookie": "^0.4.1",
-    "cross-fetch": "^3.1.4",
+    "cross-fetch": "^3.1.5",
     "ejs": "^3.1.6",
     "fast-deep-equal": "^3.1.3",
     "graphql": "^15.8.0",
-    "next-auth": "^4.1.0"
+    "next-auth": "^4.1.2"
   },
   "devDependencies": {
     "@keystone-6/core": "^1.0.1",

package/pages/NextAuthPage/dist/opensaas-keystone-nextjs-auth-pages-NextAuthPage.cjs.dev.js

@@ -3,10 +3,12 @@
 Object.defineProperty(exports, '__esModule', { value: true });
 
 var _objectSpread = require('@babel/runtime/helpers/objectSpread2');
+var _JSON$stringify = require('@babel/runtime-corejs3/core-js-stable/json/stringify');
 var NextAuth = require('next-auth');
 
 function _interopDefault (e) { return e && e.__esModule ? e : { 'default': e }; }
 
+var _JSON$stringify__default = /*#__PURE__*/_interopDefault(_JSON$stringify);
 var NextAuth__default = /*#__PURE__*/_interopDefault(NextAuth);
 
 async function findMatchingIdentity(identityField, identity, queryAPI) {
@@ -35,7 +37,7 @@ async function findMatchingIdentity(identityField, identity, queryAPI) {
   };
 }
 
-async function validateNextAuth(list, identityField, identity, protectIdentities, itemAPI) {
+async function validateNextAuth(identityField, identity, protectIdentities, itemAPI) {
   const match = await findMatchingIdentity(identityField, identity, itemAPI);
   const {
     item
@@ -85,9 +87,11 @@ function NextAuthPage(props) {
           identity = user.id;
         } else if (typeof user.id === 'number') {
           identity = user.id;
+        } else {
+          identity = 0;
         }
 
-        const result = await validateNextAuth(list, identityField, identity, protectIdentities, queryAPI);
+        const result = await validateNextAuth(identityField, identity, protectIdentities, queryAPI);
         const data = {}; // eslint-disable-next-line no-restricted-syntax
 
         for (const key in userMap) {
@@ -112,24 +116,26 @@ function NextAuthPage(props) {
 
         if (!result.success) {
           if (!autoCreate) {
-            console.log('False');
+            console.log('`autoCreate` if set to `false`, skipping user auto-creation');
             return false;
           }
 
-          console.log('Create User');
-          await list.createOne({
+          console.log('`autoCreate` if set to `true`, auto-creating a new user');
+          const createUser = await list.createOne({
             data
           }).then(returned => {
-            console.log(returned);
+            console.log('User Created', _JSON$stringify__default["default"](returned));
             return true;
           }).catch(error => {
             console.log(error);
             throw new Error(error);
           });
-        } else {
-          // await list.updateOne({where: {id: result.item.id}, data});
-          return result.success;
-        }
+          console.log('Created User', createUser);
+          return createUser;
+        } // await list.updateOne({where: {id: result.item.id}, data});
+
+
+        return result.success;
       },
 
       async redirect({
@@ -158,30 +164,29 @@ function NextAuthPage(props) {
         const identity = token.sub;
 
         if (!token.itemId) {
-          const result = await validateNextAuth(list, identityField, identity, protectIdentities, queryAPI);
+          const result = await validateNextAuth(identityField, identity, protectIdentities, queryAPI);
 
           if (!result.success) {
-            return;
+            return token;
           }
 
-          const data = await query[listKey].findOne({
-            where: {
-              id: result.item.id
-            },
-            query: sessionData || 'id'
-          });
+          token.itemId = result.item.id;
+        }
 
-          const returnToken = _objectSpread(_objectSpread({}, token), {}, {
-            data,
-            subject: token.sub,
-            listKey,
-            itemId: result.item.id.toString()
-          });
+        const data = await query[listKey].findOne({
+          where: {
+            id: token.itemId
+          },
+          query: sessionData || 'id'
+        });
 
-          return returnToken;
-        }
+        const returnToken = _objectSpread(_objectSpread({}, token), {}, {
+          data,
+          subject: token.sub,
+          listKey
+        });
 
-        return token;
+        return returnToken;
       }
 
     }

package/pages/NextAuthPage/dist/opensaas-keystone-nextjs-auth-pages-NextAuthPage.cjs.prod.js

@@ -3,10 +3,12 @@
 Object.defineProperty(exports, '__esModule', { value: true });
 
 var _objectSpread = require('@babel/runtime/helpers/objectSpread2');
+var _JSON$stringify = require('@babel/runtime-corejs3/core-js-stable/json/stringify');
 var NextAuth = require('next-auth');
 
 function _interopDefault (e) { return e && e.__esModule ? e : { 'default': e }; }
 
+var _JSON$stringify__default = /*#__PURE__*/_interopDefault(_JSON$stringify);
 var NextAuth__default = /*#__PURE__*/_interopDefault(NextAuth);
 
 async function findMatchingIdentity(identityField, identity, queryAPI) {
@@ -35,7 +37,7 @@ async function findMatchingIdentity(identityField, identity, queryAPI) {
   };
 }
 
-async function validateNextAuth(list, identityField, identity, protectIdentities, itemAPI) {
+async function validateNextAuth(identityField, identity, protectIdentities, itemAPI) {
   const match = await findMatchingIdentity(identityField, identity, itemAPI);
   const {
     item
@@ -85,9 +87,11 @@ function NextAuthPage(props) {
           identity = user.id;
         } else if (typeof user.id === 'number') {
           identity = user.id;
+        } else {
+          identity = 0;
         }
 
-        const result = await validateNextAuth(list, identityField, identity, protectIdentities, queryAPI);
+        const result = await validateNextAuth(identityField, identity, protectIdentities, queryAPI);
         const data = {}; // eslint-disable-next-line no-restricted-syntax
 
         for (const key in userMap) {
@@ -112,24 +116,26 @@ function NextAuthPage(props) {
 
         if (!result.success) {
           if (!autoCreate) {
-            console.log('False');
+            console.log('`autoCreate` if set to `false`, skipping user auto-creation');
             return false;
           }
 
-          console.log('Create User');
-          await list.createOne({
+          console.log('`autoCreate` if set to `true`, auto-creating a new user');
+          const createUser = await list.createOne({
             data
           }).then(returned => {
-            console.log(returned);
+            console.log('User Created', _JSON$stringify__default["default"](returned));
             return true;
           }).catch(error => {
             console.log(error);
             throw new Error(error);
           });
-        } else {
-          // await list.updateOne({where: {id: result.item.id}, data});
-          return result.success;
-        }
+          console.log('Created User', createUser);
+          return createUser;
+        } // await list.updateOne({where: {id: result.item.id}, data});
+
+
+        return result.success;
       },
 
       async redirect({
@@ -158,30 +164,29 @@ function NextAuthPage(props) {
         const identity = token.sub;
 
         if (!token.itemId) {
-          const result = await validateNextAuth(list, identityField, identity, protectIdentities, queryAPI);
+          const result = await validateNextAuth(identityField, identity, protectIdentities, queryAPI);
 
           if (!result.success) {
-            return;
+            return token;
           }
 
-          const data = await query[listKey].findOne({
-            where: {
-              id: result.item.id
-            },
-            query: sessionData || 'id'
-          });
+          token.itemId = result.item.id;
+        }
 
-          const returnToken = _objectSpread(_objectSpread({}, token), {}, {
-            data,
-            subject: token.sub,
-            listKey,
-            itemId: result.item.id.toString()
-          });
+        const data = await query[listKey].findOne({
+          where: {
+            id: token.itemId
+          },
+          query: sessionData || 'id'
+        });
 
-          return returnToken;
-        }
+        const returnToken = _objectSpread(_objectSpread({}, token), {}, {
+          data,
+          subject: token.sub,
+          listKey
+        });
 
-        return token;
+        return returnToken;
       }
 
     }

package/pages/NextAuthPage/dist/opensaas-keystone-nextjs-auth-pages-NextAuthPage.esm.js

@@ -1,4 +1,5 @@
 import _objectSpread from '@babel/runtime/helpers/esm/objectSpread2';
+import _JSON$stringify from '@babel/runtime-corejs3/core-js-stable/json/stringify';
 import NextAuth from 'next-auth';
 
 async function findMatchingIdentity(identityField, identity, queryAPI) {
@@ -27,7 +28,7 @@ async function findMatchingIdentity(identityField, identity, queryAPI) {
   };
 }
 
-async function validateNextAuth(list, identityField, identity, protectIdentities, itemAPI) {
+async function validateNextAuth(identityField, identity, protectIdentities, itemAPI) {
   const match = await findMatchingIdentity(identityField, identity, itemAPI);
   const {
     item
@@ -77,9 +78,11 @@ function NextAuthPage(props) {
           identity = user.id;
         } else if (typeof user.id === 'number') {
           identity = user.id;
+        } else {
+          identity = 0;
         }
 
-        const result = await validateNextAuth(list, identityField, identity, protectIdentities, queryAPI);
+        const result = await validateNextAuth(identityField, identity, protectIdentities, queryAPI);
         const data = {}; // eslint-disable-next-line no-restricted-syntax
 
         for (const key in userMap) {
@@ -104,24 +107,26 @@ function NextAuthPage(props) {
 
         if (!result.success) {
           if (!autoCreate) {
-            console.log('False');
+            console.log('`autoCreate` if set to `false`, skipping user auto-creation');
             return false;
           }
 
-          console.log('Create User');
-          await list.createOne({
+          console.log('`autoCreate` if set to `true`, auto-creating a new user');
+          const createUser = await list.createOne({
             data
           }).then(returned => {
-            console.log(returned);
+            console.log('User Created', _JSON$stringify(returned));
             return true;
           }).catch(error => {
             console.log(error);
             throw new Error(error);
           });
-        } else {
-          // await list.updateOne({where: {id: result.item.id}, data});
-          return result.success;
-        }
+          console.log('Created User', createUser);
+          return createUser;
+        } // await list.updateOne({where: {id: result.item.id}, data});
+
+
+        return result.success;
       },
 
       async redirect({
@@ -150,30 +155,29 @@ function NextAuthPage(props) {
         const identity = token.sub;
 
         if (!token.itemId) {
-          const result = await validateNextAuth(list, identityField, identity, protectIdentities, queryAPI);
+          const result = await validateNextAuth(identityField, identity, protectIdentities, queryAPI);
 
           if (!result.success) {
-            return;
+            return token;
           }
 
-          const data = await query[listKey].findOne({
-            where: {
-              id: result.item.id
-            },
-            query: sessionData || 'id'
-          });
+          token.itemId = result.item.id;
+        }
 
-          const returnToken = _objectSpread(_objectSpread({}, token), {}, {
-            data,
-            subject: token.sub,
-            listKey,
-            itemId: result.item.id.toString()
-          });
+        const data = await query[listKey].findOne({
+          where: {
+            id: token.itemId
+          },
+          query: sessionData || 'id'
+        });
 
-          return returnToken;
-        }
+        const returnToken = _objectSpread(_objectSpread({}, token), {}, {
+          data,
+          subject: token.sub,
+          listKey
+        });
 
-        return token;
+        return returnToken;
       }
 
     }

package/src/index.ts

@@ -292,6 +292,10 @@ export function createAuth<GeneratedListTypes extends BaseListTypeInfo>({
       lists: {
         ...keystoneConfig.lists,
       },
+      experimental: {
+        ...keystoneConfig.experimental,
+        generateNodeAPI: true,
+      },
       extendGraphqlSchema: existingExtendGraphQLSchema
         ? (schema) => existingExtendGraphQLSchema(extendGraphqlSchema(schema))
         : extendGraphqlSchema,

package/src/lib/validateNextAuth.ts

@@ -3,7 +3,6 @@ import { NextAuthErrorCode } from '../types';
 import { findMatchingIdentity } from './findMatchingIdentity';
 
 export async function validateNextAuth(
-  list: any,
   identityField: string,
   identity: string | number,
   protectIdentities: boolean,

package/src/pages/NextAuthPage.tsx

@@ -45,15 +45,16 @@ export default function NextAuthPage(props: NextAuthPageProps) {
           identity = user.id;
         } else if (typeof user.id === 'number') {
           identity = user.id;
+        } else {
+          identity = 0;
         }
         const result = await validateNextAuth(
-          list,
           identityField,
           identity,
           protectIdentities,
           queryAPI
         );
-        const data = {};
+        const data: any = {};
         // eslint-disable-next-line no-restricted-syntax
         for (const key in userMap) {
           if (Object.prototype.hasOwnProperty.call(userMap, key)) {
@@ -75,27 +76,30 @@ export default function NextAuthPage(props: NextAuthPageProps) {
 
         if (!result.success) {
           if (!autoCreate) {
-            console.log('False');
+            console.log(
+              '`autoCreate` if set to `false`, skipping user auto-creation'
+            );
             return false;
           }
-          console.log('Create User');
+          console.log(
+            '`autoCreate` if set to `true`, auto-creating a new user'
+          );
 
-          await list
+          const createUser = await list
             .createOne({ data })
             .then((returned) => {
-              console.log(returned);
-
+              console.log('User Created', JSON.stringify(returned));
               return true;
             })
             .catch((error) => {
               console.log(error);
-
               throw new Error(error);
             });
-        } else {
-          // await list.updateOne({where: {id: result.item.id}, data});
-          return result.success;
+          console.log('Created User', createUser);
+          return createUser;
         }
+        // await list.updateOne({where: {id: result.item.id}, data});
+        return result.success;
       },
       async redirect({ url }) {
         return url;
@@ -111,10 +115,9 @@ export default function NextAuthPage(props: NextAuthPageProps) {
         return returnSession;
       },
       async jwt({ token }) {
-        const identity = token.sub;
+        const identity = token.sub as number | string;
         if (!token.itemId) {
           const result = await validateNextAuth(
-            list,
             identityField,
             identity,
             protectIdentities,
@@ -122,24 +125,22 @@ export default function NextAuthPage(props: NextAuthPageProps) {
           );
 
           if (!result.success) {
-            return;
+            return token;
           }
-
-          const data = await query[listKey].findOne({
-            where: { id: result.item.id },
-            query: sessionData || 'id',
-          });
-          const returnToken = {
-            ...token,
-            data,
-            subject: token.sub,
-            listKey,
-            itemId: result.item.id.toString(),
-          };
-
-          return returnToken;
+          token.itemId = result.item.id;
         }
-        return token;
+        const data = await query[listKey].findOne({
+          where: { id: token.itemId },
+          query: sessionData || 'id',
+        });
+        const returnToken = {
+          ...token,
+          data,
+          subject: token.sub,
+          listKey,
+        };
+
+        return returnToken;
       },
     },
   });

package/src/types.ts

@@ -12,7 +12,7 @@ export type AuthGqlNames = {
 
 export type NextAuthSession = { listKey: string; itemId: string; data: any };
 
-export type NextAuthProviders = [Provider];
+export type NextAuthProviders = Provider[];
 
 type KeytoneAuthProviders = {
   providers: NextAuthProviders;