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

URL check is too restrictive

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.6, 1.6.1, 1.6.2, 1.6.3, 1.7
    • 1.6.4, 1.7.1, 1.8
    • Unknown
    • None
    • All
    • MOODLE_16_STABLE, MOODLE_17_STABLE
    • MOODLE_16_STABLE, MOODLE_17_STABLE, MOODLE_18_STABLE

    Description

      It is impossible to enter URL with a port number (e.g. http://www.apple.com/main/rss/hotnews/hotnews.rss will work, but not http://www.apple.com:80/main/rss/hotnews/hotnews.rss)

      This happens when a call to clean_param(..., PARAM_URL) is made and the url parameter performed with validateUrlSyntax(). Currently, it does prevent the port number.

      Find below the diff to apply to have the port made available:

      443c443
      < if (!empty($param) && validateUrlSyntax($param, 's?H?S?F?E?u-P-a?I?p?f?q?r?')) {

      > if (!empty($param) && validateUrlSyntax($param, 's?H?S?F?E?u-P-a?I?p-f?q?r?')) {

      Attachments

        Activity

          People

            skodak Petr Skoda
            hugues.pisapia Hugues Pisapia
            Nobody Nobody
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Laurent David, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:
              17/Jan/07