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

unoconv uses temporary request dir which makes it hard to offload processing

    XMLWordPrintable

    Details

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

      Make sure unoconv is installed properly and if it is in a non-standard location - the path is set in config.php.

      Run ./vendor/bin/phpunit lib/tests/unoconv_test.php

      Show
      Make sure unoconv is installed properly and if it is in a non-standard location - the path is set in config.php. Run ./vendor/bin/phpunit lib/tests/unoconv_test.php
    • Affected Branches:
      MOODLE_31_STABLE
    • Fixed Branches:
      MOODLE_31_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-54544-master

      Description

      Some sites may want to use a wrapper script for unoconv to force it to always perform processing on a node that is not a webserving node. This would be fairly simple - except that we use a request only dir for file conversions which would need to be mounted on both servers.

      Instead we should use make_temp_directory and clean up after ourselves. This will have no performance impact on single node servers, and is desirable for clustered sites.

      This is all in lib/filestorage/file_storage.php

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                23/May/16