Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-58615

Remove YUI CDN support

    XMLWordPrintable

Details

    • 1
    • Team Hedgehog Sprint 1.3

    Description

      While working on MDL-54901 I faced various (unrelated!) problems with some tests instantiating the page_requirements_manager.

      There, external yui loading ($CFG->useexternalyui) is beind "manually" disabled with comment:

                  // On HTTPS sites all JS must be loaded from https sites,
                  // YUI CDN does not support https yet, sorry.
                  $CFG->useexternalyui = 0;
      

      But I've seen this one available/used in a number of places:

      https://yui-s.yahooapis.com

      So this is just to concile the comment and the reality. Note I haven't looked if that URL is official, supported, works or not... just found it looking for "yui ssl cdn" out there (drupal, stackoverflow...).

      Also, somehow related... I've seen yui stuff being served from clouflare cdn and/or goggle too...

      Anyway, issue created, I'm happy with any outcome, just wanted to be sure the comment and behaviors are correct.

      Attachments

        Issue Links

          Activity

            People

              dobedobedoh Andrew Lyons
              stronk7 Eloy Lafuente (stronk7)
              Huong Nguyen Huong Nguyen
              Paul Holden Paul Holden
              Kim Jared Lucas Kim Jared Lucas
              Amy Groshek, David Scotson, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo, David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo
              Votes:
              1 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                24/Apr/23

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 hour, 30 minutes
                  1h 30m