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

Langimport makes bad assumption about http headers

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 1.8
    • Fix Version/s: 1.8
    • Component/s: Administration
    • Labels:
      None
    • Affected Branches:
      MOODLE_18_STABLE
    • Fixed Branches:
      MOODLE_18_STABLE

      Description

      The language import function makes a bad assumption about http headers when going through a proxy only being 12 lines long, when in reality they could be longer or shorter...

      In my case I discovered this as it imported a blank line as an available language, but it could perhaps interpret a http header line as a language in a different scenario.

      My solution is to scan through the file until you find a CRLF, as per RFC1945, rather than just counting to 12.

      This guarantees we're at the start of the body (assuming a standards compliant webserver )

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              pj Jon Papaioannou added a comment -

              Fixed in STABLE and HEAD with Dan's patch.

              Show
              pj Jon Papaioannou added a comment - Fixed in STABLE and HEAD with Dan's patch.

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    31/Mar/07