Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Portfolio
    • Labels:
      None
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE

      Description

      timeouts are causing brokenness. investigate fixing the boxnet class (in box.net repo plugin) to deal with timeouts and report errors more gracefully.

        Gliffy Diagrams

          Activity

          Hide
          mjollnir Penny Leach added a comment -

          dongsheng - I'm adding you as a watcher because I think this affects you too.

          Show
          mjollnir Penny Leach added a comment - dongsheng - I'm adding you as a watcher because I think this affects you too.
          Hide
          dongsheng Dongsheng Cai added a comment -

          Penny, these is no callback function option for curl timeout, so I assume it is time-out if no return value from curl_exec, a moodle_exception will be thrown, does this make sense?

          Show
          dongsheng Dongsheng Cai added a comment - Penny, these is no callback function option for curl timeout, so I assume it is time-out if no return value from curl_exec, a moodle_exception will be thrown, does this make sense?
          Hide
          mjollnir Penny Leach added a comment -

          fine. I think it should be caught in the boxlib class though and the error status set there, rather than falling through to whatever is calling boxlib functions.

          Show
          mjollnir Penny Leach added a comment - fine. I think it should be caught in the boxlib class though and the error status set there, rather than falling through to whatever is calling boxlib functions.
          Hide
          mjollnir Penny Leach added a comment -

          closing. DS added extra stuff in the boxnet client class.

          Show
          mjollnir Penny Leach added a comment - closing. DS added extra stuff in the boxnet client class.

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                24/Nov/10