Merge pull request #833 from catalyst/fix-connectivity-check

feat: ignore cors during connectivity check

Author: Peterburnett

amd/build/connectivity_test.min.js

@@ -23,10 +23,13 @@
  */
 
 export const init = (checkTarget, redirURL) => {
-    const http = new XMLHttpRequest();
-    http.open('HEAD', checkTarget);
-    http.addEventListener('load', () => {
+    // We need to use no-cors to ignore cors, however,
+    // this means we are returned an opaque response.
+    // But an opaque response is info to say if the site is accessible or not.
+    fetch(checkTarget, { mode: 'no-cors', method: 'HEAD' })
+    .then(() => {
         window.location = redirURL;
-    });
-    http.send();
+    })
+    // Do nothing with error, we don't care about it.
+    .catch(() => {});
 };

amd/build/connectivity_test.min.js.map

@@ -24,8 +24,8 @@
 
 defined('MOODLE_INTERNAL') || die();
 
-$plugin->version   = 2024082000;    // The current plugin version (Date: YYYYMMDDXX).
-$plugin->release   = 2024082000;    // Match release exactly to version.
+$plugin->version   = 2024090900;    // The current plugin version (Date: YYYYMMDDXX).
+$plugin->release   = 2024090900;    // Match release exactly to version.
 $plugin->requires  = 2017051509;    // Requires PHP 7, 2017051509 = T12. M3.3
                                     // Strictly we require either Moodle 3.5 OR
                                     // we require Totara 3.3, but the version number