Moodle
  1. Moodle
  2. MDL-36950

Automatic update deployment does not support sites behind a proxy

        Details

        • Type: Bug Bug
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.4
        • Fix Version/s: 2.4
        • Component/s: Administration
        • Labels:
        • Testing Instructions:
          Hide

          Testing difficulty: MEDIUM (requires a site behind a proxy)

          1. Prepare a site behind a proxy and make sure the proxy is defined at Site administration ► Server ► HTTP.
          2. TEST: Make sure that you are able to update a plugin via "Install this update" button.
          3. TEST (optional but preferred): Use a monitoring tool to make sure that the traffic really goes thru the proxy (I was using http://www.charlesproxy.com/ during the development)
          Show
          Testing difficulty: MEDIUM (requires a site behind a proxy) Prepare a site behind a proxy and make sure the proxy is defined at Site administration ► Server ► HTTP. TEST: Make sure that you are able to update a plugin via "Install this update" button. TEST (optional but preferred): Use a monitoring tool to make sure that the traffic really goes thru the proxy (I was using http://www.charlesproxy.com/ during the development)
        • Affected Branches:
          MOODLE_24_STABLE
        • Fixed Branches:
          MOODLE_24_STABLE
        • Pull from Repository:
        • Pull Master Branch:
          MDL-36950-updates-proxy
        • Rank:
          46491

          Description

          Shortly, proxy related settings ($CFG->proxyhost and friends) are not passed to mdeploy.php. They should be.

            Issue Links

            has a non-specific relationship to

            New Feature - A new Moodle feature which has yet to be developed. MDL-35238 Automatic plugin update deployment

            • Major - Major loss of function, incorrect output
            • Closed
            is blocked by

            Improvement - An enhancement to an existing Moodle feature. MDL-36903 Download pre-check for plugin ZIP packages

            • Minor - Minor loss of function where workaround is possible
            • Closed
            testing discovered

            Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-36973 curl class doesn't respect is_proxybypass

            • Minor - Minor loss of function where workaround is possible
            • Open

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              David Mudrak added a comment -

              DEAR INTEGRATORS,

              Please integrate MDL-36903 first. This branch is built on top of it.

              Show
              David Mudrak added a comment - DEAR INTEGRATORS, Please integrate MDL-36903 first. This branch is built on top of it.
              Hide
              Permalink
              David Mudrak added a comment - 
              The following changes since commit 333048e06b19c06828ea3d30f63ecc5a4a0088df:

  MDL-36903 Add a link to Moodle documentation from mdeploy error pages (2012-11-30 16:09:58 +0100)

are available in the git repository at:
  git://github.com/mudrd8mz/moodle.git MDL-36950-updates-proxy

David Mudrák (2):
      MDL-36950 Make mdeploy.php accept proxy related settings
      MDL-36950 Pass the Moodle proxy setting to the mdeploy.php utility

 lib/pluginlib.php |   22 ++++++++++++++++++++++
 mdeploy.php       |   24 ++++++++++++++++++++++++
 2 files changed, 46 insertions(+), 0 deletions(-)
              Show
              David Mudrak added a comment - The following changes since commit 333048e06b19c06828ea3d30f63ecc5a4a0088df: MDL-36903 Add a link to Moodle documentation from mdeploy error pages (2012-11-30 16:09:58 +0100) are available in the git repository at: git: //github.com/mudrd8mz/moodle.git MDL-36950-updates-proxy David Mudrák (2): MDL-36950 Make mdeploy.php accept proxy related settings MDL-36950 Pass the Moodle proxy setting to the mdeploy.php utility lib/pluginlib.php | 22 ++++++++++++++++++++++ mdeploy.php | 24 ++++++++++++++++++++++++ 2 files changed, 46 insertions(+), 0 deletions(-)
              Hide
              Permalink
              Dan Poltawski added a comment -

              We need an issue for the is_proxybypass() problem, else we'll never fix it

              Show
              Dan Poltawski added a comment - We need an issue for the is_proxybypass() problem, else we'll never fix it
              Hide
              Permalink
              Dan Poltawski added a comment -

              I have created MDL-36973 for that.

              Show
              Dan Poltawski added a comment - I have created MDL-36973 for that.
              Hide
              Permalink
              Dan Poltawski added a comment -

              I've integrated this now David.

              Show
              Dan Poltawski added a comment - I've integrated this now David.
              Hide
              Permalink
              Dan Poltawski added a comment -

              Tested during integration, and looks good!

              Show
              Dan Poltawski added a comment - Tested during integration, and looks good!
              Hide
              Permalink
              Eloy Lafuente (stronk7) added a comment -

              Just in time for Moodle 2.4.0 release, thanks!

              Closing, ciao

              Show
              Eloy Lafuente (stronk7) added a comment - Just in time for Moodle 2.4.0 release, thanks! Closing, ciao

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: