Bump nvmrc back down to 14 and update comments

.nvmrc

@@ -1 +1 @@
-15
+14

packages/remix-node/globals.ts

@@ -4,9 +4,9 @@ import { Headers, Request, Response, fetch } from "./fetch";
 
 declare global {
   namespace NodeJS {
-    type GlobalSignFunc = (data: string, secret: string) => Promise<string>;
+    type GlobalSignFunc = (value: string, secret: string) => Promise<string>;
     type GlobalUnsignFunc = (
-      encrypted: string,
+      signed: string,
       secret: string
     ) => Promise<string | false>;
 
@@ -18,8 +18,8 @@ declare global {
       Response: typeof Response;
       fetch: typeof fetch;
 
-      // TODO: Once node v15 hits LTS we should remove these globals and provide
-      // the webcrypto API instead.
+      // TODO: Once node v16 is available on AWS we should remove these globals
+      // and provide the webcrypto API instead.
       sign: GlobalSignFunc;
       unsign: GlobalUnsignFunc;
     }

packages/remix-node/sessions/fileStorage.ts

@@ -37,6 +37,8 @@ export function createFileSessionStorage({
       let content = JSON.stringify({ data, expires });
 
       while (true) {
+        // TODO: Once node v16 is available on AWS we should use the webcrypto
+        // API's crypto.getRandomValues() function here instead.
         let randomBytes = crypto.randomBytes(8);
         // This storage manages an id space of 2^64 ids, which is far greater
         // than the maximum number of files allowed on an NTFS or ext4 volume

packages/remix-server-runtime/cookieSigning.ts

@@ -1,6 +1,6 @@
-// TODO: Once node v15 hits LTS we should use the globally provided webcrypto "crypto"
-// variable and re-enable this code-path in "./cookies.ts" instead of referencing the
-// sign and unsign globals.
+// TODO: Once node v16 is available on AWS we should use the globally provided
+// webcrypto "crypto" variable and re-enable this code-path in "./cookies.ts"
+// instead of referencing the sign and unsign globals.
 
 // const encoder = new TextEncoder();
 

packages/remix-server-runtime/cookies.ts

@@ -1,7 +1,9 @@
 import type { CookieParseOptions, CookieSerializeOptions } from "cookie";
 import { parse, serialize } from "cookie";
 
-// TODO: Once node v15 hits LTS `import { sign, unsign } from "./cookieSigning";`
+// TODO: Once node v16 is available on AWS we should use these instead of the
+// global `sign` and `unsign` functions.
+//import { sign, unsign } from "./cookieSigning";
 
 export type { CookieParseOptions, CookieSerializeOptions };