chore: upgrade apollo to v4 and fix related issues (#429)

* chore: upgrade apollo to v4 and fix related issues
* chore: upgrade node

Author: vnugent

.github/workflows/nodejs.yml

@@ -119,7 +119,7 @@ jobs:
       - name: Setup Node.js environment
         uses: actions/setup-node@v3
         with:
-          node-version: '16.15'
+          node-version-file: 'package.json'
           cache: 'yarn'
 
       - name: Start MongoDB

Dockerfile

@@ -1,4 +1,4 @@
-FROM registry.gitlab.com/openbeta/openbeta-nodejs-docker:16
+FROM registry.gitlab.com/openbeta/openbeta-nodejs-docker:18
 
 ENV APP_DIR=/apps/openbeta-graphql
 

package.json

@@ -14,17 +14,18 @@
     "@types/underscore": "^1.11.4",
     "cross-env": "^7.0.3",
     "husky": "^8.0.1",
-    "jest": "29.4.2",
-    "jest-extended": "^3.2.3",
-    "mongodb-memory-server": "^8.12.2",
+    "jest": "^29.7.0",
+    "jest-extended": "^4.0.2",
+    "mongodb-memory-server": "^10.1.2",
     "nock": "^13.3.0",
     "supertest": "^6.3.3",
-    "ts-jest": "^29.0.5",
+    "ts-jest": "^29.2.5",
     "ts-standard": "^12.0.0",
     "typescript": "4.9.5",
     "wait-for-expect": "^3.0.2"
   },
   "dependencies": {
+    "@apollo/server": "^4.11.2",
     "@babel/runtime": "^7.17.2",
     "@google-cloud/storage": "^6.9.5",
     "@graphql-tools/schema": "^8.3.1",
@@ -36,9 +37,7 @@
     "@turf/convex": "^6.5.0",
     "@turf/helpers": "^6.5.0",
     "@types/uuid": "^8.3.3",
-    "apollo-datasource-mongodb": "^0.5.4",
-    "apollo-server-core": "^3.13.0",
-    "apollo-server-express": "^3.13.0",
+    "apollo-datasource-mongodb": "^0.6.0",
     "auth0": "^3.4.0",
     "axios": "^1.3.6",
     "body-parser": "^1.20.2",
@@ -48,26 +47,27 @@
     "dotenv": "^16.4.4",
     "express": "^4.18.2",
     "glob": "^10.2.2",
-    "graphql": "^16.8.1",
+    "graphql": "^16.9.0",
     "graphql-middleware": "^6.1.31",
     "graphql-shield": "^7.5.0",
+    "graphql-tag": "^2.12.6",
     "graphql-type-json": "^0.3.2",
     "i18n-iso-countries": "^7.5.0",
     "immer": "^9.0.15",
     "jsonwebtoken": "^8.5.1",
     "jwks-rsa": "^2.1.4",
-    "mongoose": "6.4.7",
-    "mongoose-lean-virtuals": "^0.9.1",
+    "mongoose": "^7.8.3",
+    "mongoose-lean-virtuals": "^1.0.0",
     "node-fetch": "2",
     "p-limit": "^4.0.0",
-    "pino": "^8.2.0",
+    "pino": "^9.5.0",
     "pino-logflare": "^0.4.2",
     "sanitize-html": "^2.7.2",
     "sharp": "^0.32.0",
-    "typesense": "^1.2.1",
+    "typesense": "^1.8.2",
     "underscore": "^1.13.2",
     "uuid": "^8.3.2",
-    "uuid-mongodb": "^2.5.1",
+    "uuid-mongodb": "^2.6.0",
     "yup": "^1.1.1"
   },
   "scripts": {
@@ -109,6 +109,6 @@
   },
   "type": "module",
   "engines": {
-    "node": ">=16.14.0"
+    "node": ">=18.20.0 <19"
   }
 }

src/__tests__/areas.ts

@@ -1,4 +1,4 @@
-import { ApolloServer } from 'apollo-server-express'
+import { ApolloServer } from '@apollo/server'
 import muuid from 'uuid-mongodb'
 import { jest } from '@jest/globals'
 import MutableAreaDataSource from '../model/MutableAreaDataSource.js'

src/__tests__/bulkImport.test.ts

@@ -1,4 +1,4 @@
-import {ApolloServer} from "apollo-server-express";
+import {ApolloServer} from "@apollo/server";
 import muuid from "uuid-mongodb";
 import express from "express";
 import {InMemoryDB} from "../utils/inMemoryDB.js";
@@ -49,7 +49,6 @@ describe('bulkImportAreas', () => {
     // "59f1d95a-627d-4b8c-91b9-389c7424cb54" instead of base64 "WfHZWmJ9S4yRuTicdCTLVA==".
     user = muuid.mode('relaxed').v4()
     userUuid = muuidToString(user)
-
     bulkImport = BulkImportDataSource.getInstance()
     climbs = MutableClimbDataSource.getInstance()
   })

src/__tests__/history.ts

@@ -1,4 +1,4 @@
-import { ApolloServer } from 'apollo-server-express'
+import { ApolloServer } from '@apollo/server'
 import muuid from 'uuid-mongodb'
 import { jest } from '@jest/globals'
 import MutableAreaDataSource from '../model/MutableAreaDataSource.js'

src/__tests__/organizations.ts

@@ -1,10 +1,10 @@
-import { ApolloServer } from 'apollo-server-express'
+import { ApolloServer } from '@apollo/server'
 import muuid from 'uuid-mongodb'
 import MutableAreaDataSource from '../model/MutableAreaDataSource.js'
 import MutableOrganizationDataSource from '../model/MutableOrganizationDataSource.js'
 import { AreaType } from '../db/AreaTypes.js'
 import { OperationType, OrganizationEditableFieldsType, OrganizationType, OrgType } from '../db/OrganizationTypes.js'
-import { changelogDataSource } from '../model/ChangeLogDataSource.js'
+import ChangeLogDataSource from '../model/ChangeLogDataSource.js'
 import { queryAPI, setUpServer } from '../utils/testUtils.js'
 import { muuidToString } from '../utils/helpers.js'
 import { validate as validateMuuid } from 'uuid'
@@ -142,7 +142,7 @@ describe('organizations API', () => {
       // eslint-disable-next-line
       await new Promise(res => setTimeout(res, 1000))
 
-      const orgHistory = await changelogDataSource.getOrganizationChangeSets()
+      const orgHistory = await ChangeLogDataSource.getInstance().getOrganizationChangeSets()
       expect(orgHistory).toHaveLength(2)
 
       // verify changes in most recent order

src/__tests__/ticks.ts

@@ -1,4 +1,4 @@
-import { ApolloServer } from 'apollo-server-express'
+import { ApolloServer } from '@apollo/server'
 import muuid from 'uuid-mongodb'
 import { jest } from '@jest/globals'
 import { queryAPI, setUpServer } from '../utils/testUtils.js'
@@ -211,7 +211,7 @@ describe('ticks API', () => {
           }
         },
         userUuid,
-        roles: ['user_admin'],
+        roles: [], // ['user_admin'],
         app
       })
 

src/auth/local-dev/middleware.ts

@@ -1,21 +1,19 @@
-/*
-* This file is a mod of src/auth/middleware.ts and is used when starting the server via `yarn serve-dev`
-* It bypasses the authentication for local development
-*/
 import muuid, { MUUID } from 'uuid-mongodb'
 import { AuthUserType } from '../../types.js'
 import { logger } from '../../logger.js'
 
-export const localDevBypassAuthContext = (() => {
-  const testUUID: MUUID = muuid.v4()
+const testUUID: MUUID = muuid.v4()
 
-  return async ({ req }): Promise<any> => {
-    const user: AuthUserType = {
-      roles: ['user_admin', 'org_admin', 'editor'],
-      uuid: testUUID,
-      isBuilder: false
-    }
-    logger.info(`The user.roles for this session is: ${user.roles.toString()}`)
-    return { user }
+/*
+* This file is a mod of src/auth/middleware.ts and is used when starting the server via `yarn serve-dev`
+* It bypasses the authentication for local development
+*/
+export const localDevBypassAuthContext = async ({ req }): Promise<any> => {
+  const user: AuthUserType = {
+    roles: ['user_admin', 'org_admin', 'editor'],
+    uuid: testUUID,
+    isBuilder: false
   }
-})()
+  logger.info(`The user.roles for this session is: ${user.roles.toString()}`)
+  return { user }
+}

src/auth/middleware.ts

@@ -1,8 +1,14 @@
 import muid from 'uuid-mongodb'
+import { Request } from 'express'
 import { AuthUserType } from '../types.js'
 import { verifyJWT } from './util.js'
 import { logger } from '../logger.js'
 
+export interface CustomContext {
+  user: AuthUserType
+  token?: string
+}
+
 const EMTPY_USER: AuthUserType = {
   isBuilder: false,
   roles: [],
@@ -12,7 +18,7 @@ const EMTPY_USER: AuthUserType = {
 /**
  * Create a middleware context for Apollo server
  */
-export const createContext = async ({ req }): Promise<any> => {
+export const createContext = async ({ req }: { req: Request }): Promise<CustomContext> => {
   try {
     return await validateTokenAndExtractUser(req)
   } catch (e) {
@@ -21,7 +27,7 @@ export const createContext = async ({ req }): Promise<any> => {
   }
 }
 
-async function validateTokenAndExtractUser (req: Request): Promise<{ user: AuthUserType, token?: string }> {
+async function validateTokenAndExtractUser (req: Request): Promise<CustomContext> {
   const { headers } = req
   // eslint-disable-next-line @typescript-eslint/dot-notation
   const authHeader = String(headers?.['authorization'] ?? '')