fix: add warning about setting both useCdn and withCredentials to true (

#849)

* fix: add warning about setting both useCdn and withCredentials to true

* fix: Remove semicolon and update warning message

* chore: fix prettier warning

Signed-off-by: Cody Olsen <81981+stipsan@users.noreply.github.com>

* chore: fix prettier (again)

Signed-off-by: Cody Olsen <81981+stipsan@users.noreply.github.com>

---------

Signed-off-by: Cody Olsen <81981+stipsan@users.noreply.github.com>
Co-authored-by: Cody Olsen <81981+stipsan@users.noreply.github.com>

src/config.ts

@@ -130,6 +130,11 @@ export const initConfig = (
 
   newConfig.apiVersion = `${newConfig.apiVersion}`.replace(/^v/, '')
   newConfig.isDefaultApi = newConfig.apiHost === defaultConfig.apiHost
+
+  if (newConfig.useCdn === true && newConfig.withCredentials) {
+    warnings.printCdnAndWithCredentialsWarning()
+  }
+
   // If `useCdn` is undefined, we treat it as `true`
   newConfig.useCdn = newConfig.useCdn !== false && !newConfig.withCredentials
 

src/warnings.ts

@@ -6,6 +6,11 @@ const createWarningPrinter = (message: string[]) =>
   // eslint-disable-next-line no-console
   once((...args: Any[]) => console.warn(message.join(' '), ...args))
 
+export const printCdnAndWithCredentialsWarning = createWarningPrinter([
+  `Because you set \`withCredentials\` to true, we will override your \`useCdn\``,
+  `setting to be false since (cookie-based) credentials are never set on the CDN`,
+])
+
 export const printCdnWarning = createWarningPrinter([
   `Since you haven't set a value for \`useCdn\`, we will deliver content using our`,
   `global, edge-cached API-CDN. If you wish to have content delivered faster, set`,