Moodle
  1. Moodle
  2. MDL-41122

Remote RSS block fails on a valid Atom feed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.3.8, 2.4.5, 2.5.1
    • Fix Version/s: 2.4.6, 2.5.2
    • Component/s: RSS
    • Labels:
    • Rank:
      52059

      Description

      Remote RSS block fails on the following URL:
      http://blog.une.edu.au/serviceannounce/feed/

      Error message:
      "Error loading this RSS feed (This XML document is invalid, likely due to invalid characters. XML error: Reserved XML Name at line 2, column 38)"

      The URL is deemed valid by the validator:
      http://validator.w3.org/feed/check.cgi?url=http%3A%2F%2Fblog.une.edu.au%2Fserviceannounce%2Ffeed%2F

      Investigation is showing that this is happening because the RSS body sent to Simplepie after extraction of headers begins with an empty line.

      Solution: trim() the RSS body before sending to Simplepie.

      TEST INSTRUCTIONS:
      ==================
      1. Navigate to a test course.
      2. Add the Remote RSS Feeds block.
      3. Configure the block with the following URL: http://blog.une.edu.au/serviceannounce/feed/
      4. Upon save, the block should add the URL successfully and not issue an "Error loading this RSS feed" message.

        Issue Links

          Activity

          Hide
          Dan Poltawski added a comment -

          Thanks Kirill, i'm attaching a copy of the current output of that feed so that it can be used for testing (in case that feed goes away)

          I haven't tested it though.

          Show
          Dan Poltawski added a comment - Thanks Kirill, i'm attaching a copy of the current output of that feed so that it can be used for testing (in case that feed goes away) I haven't tested it though.
          Hide
          Kirill Astashov added a comment - - edited

          Added 2.3, 2.4, 2.5 and master diff and pull URLs.

          Show
          Kirill Astashov added a comment - - edited Added 2.3, 2.4, 2.5 and master diff and pull URLs.
          Hide
          Dan Poltawski added a comment -

          Thanks for the perfect testing instructions and patch Kirill, looks perfect and i'm sending it for integration.

          Show
          Dan Poltawski added a comment - Thanks for the perfect testing instructions and patch Kirill, looks perfect and i'm sending it for integration.
          Hide
          Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Dan Poltawski added a comment -

          The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

          TIA and ciao

          Show
          Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
          Hide
          Damyon Wiese added a comment -

          Thanks Kirill - looks good to me.

          Integrated to 24, 25 and master.

          I tested on master and it fails on that feed url without the patch and passes with it.

          Show
          Damyon Wiese added a comment - Thanks Kirill - looks good to me. Integrated to 24, 25 and master. I tested on master and it fails on that feed url without the patch and passes with it.
          Hide
          Mark Nelson added a comment -

          Works as expected, passing. Thanks for your contribution to Moodle, Kirill.

          Show
          Mark Nelson added a comment - Works as expected, passing. Thanks for your contribution to Moodle, Kirill.
          Hide
          Damyon Wiese added a comment -

          Thanks for your efforts. This issue is one of the outstanding issues that passed all our testing and was accepted into Moodle this week.

          Hurray!

          Show
          Damyon Wiese added a comment - Thanks for your efforts. This issue is one of the outstanding issues that passed all our testing and was accepted into Moodle this week. Hurray!

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: