@netlify/config 20.21.3 → 20.21.4

package/lib/api/site_info.d.ts

@@ -20,7 +20,7 @@ type GetSiteInfoOpts = {
  * Silently ignore API errors. For example the network connection might be down,
  * but local builds should still work regardless.
  */
-export declare const getSiteInfo: ({ api, siteId, accountId, mode, context, offline, testOpts, siteFeatureFlagPrefix, }: GetSiteInfoOpts) => Promise<{
+export declare const getSiteInfo: ({ api, siteId, accountId, mode, context, offline, testOpts, siteFeatureFlagPrefix, featureFlags, }: GetSiteInfoOpts) => Promise<{
     siteInfo: any;
     accounts: any;
     addons: any;

package/lib/api/site_info.js

@@ -11,22 +11,25 @@ import { ERROR_CALL_TO_ACTION } from '../log/messages.js';
  * Silently ignore API errors. For example the network connection might be down,
  * but local builds should still work regardless.
  */
-export const getSiteInfo = async function ({ api, siteId, accountId, mode, context, offline = false, testOpts = {}, siteFeatureFlagPrefix, }) {
+export const getSiteInfo = async function ({ api, siteId, accountId, mode, context, offline = false, testOpts = {}, siteFeatureFlagPrefix, featureFlags = {}, }) {
     const { env: testEnv = false } = testOpts;
+    const errorOnExtensionFetchFail = featureFlags.error_builds_on_extension_fetch_fail;
     if (api === undefined || mode === 'buildbot' || testEnv) {
         const siteInfo = {};
         if (siteId !== undefined)
             siteInfo.id = siteId;
         if (accountId !== undefined)
             siteInfo.account_id = accountId;
-        const integrations = mode === 'buildbot' && !offline ? await getIntegrations({ siteId, testOpts, offline, accountId }) : [];
+        const integrations = mode === 'buildbot' && !offline
+            ? await getIntegrations({ siteId, testOpts, offline, accountId, errorOnExtensionFetchFail })
+            : [];
         return { siteInfo, accounts: [], addons: [], integrations };
     }
     const promises = [
         getSite(api, siteId, siteFeatureFlagPrefix),
         getAccounts(api),
         getAddons(api, siteId),
-        getIntegrations({ siteId, testOpts, offline, accountId }),
+        getIntegrations({ siteId, testOpts, offline, accountId, errorOnExtensionFetchFail }),
     ];
     const [siteInfo, accounts, addons, integrations] = await Promise.all(promises);
     if (siteInfo.use_envelope) {
@@ -68,7 +71,7 @@ const getAddons = async function (api, siteId) {
         throwUserError(`Failed retrieving addons for site ${siteId}: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
     }
 };
-const getIntegrations = async function ({ siteId, accountId, testOpts, offline, }) {
+const getIntegrations = async function ({ siteId, accountId, testOpts, offline, errorOnExtensionFetchFail, }) {
     if (!siteId || offline) {
         return [];
     }
@@ -78,14 +81,29 @@ const getIntegrations = async function ({ siteId, accountId, testOpts, offline,
     const url = accountId
         ? `${baseUrl}team/${accountId}/integrations/installations/meta/${siteId}`
         : `${baseUrl}site/${siteId}/integrations/safe`;
+    if (errorOnExtensionFetchFail) {
+        try {
+            const response = await fetch(url);
+            if (!response.ok) {
+                throw new Error(`Unexpected status code ${response.status} from fetching extensions`);
+            }
+            const bodyText = await response.text();
+            if (bodyText === '') {
+                return [];
+            }
+            const integrations = await JSON.parse(bodyText);
+            return Array.isArray(integrations) ? integrations : [];
+        }
+        catch (error) {
+            return throwUserError(`Failed retrieving extensions for site ${siteId}: ${error.message}. ${ERROR_CALL_TO_ACTION}`);
+        }
+    }
     try {
         const response = await fetch(url);
         const integrations = await response.json();
         return Array.isArray(integrations) ? integrations : [];
     }
     catch (error) {
-        // TODO: We should consider blocking the build as integrations are a critical part of the build process
-        // https://linear.app/netlify/issue/CT-1214/implement-strategy-in-builds-to-deal-with-integrations-that-we-fail-to
         return [];
     }
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@netlify/config",
-  "version": "20.21.3",
+  "version": "20.21.4",
   "description": "Netlify config module",
   "type": "module",
   "exports": "./lib/index.js",
@@ -95,5 +95,5 @@
   "engines": {
     "node": "^14.16.0 || >=16.0.0"
   },
-  "gitHead": "0a66a5592b80f8b2e2538ebc385d3eea8777c7b7"
+  "gitHead": "04134491e36ab1b6ccf5990edf1885e43d8a15a0"
 }