Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-21825

Temporary files not always deleted when copying from DFS to FS



      In eZDFSFileHandlerMySQLiBackend::_fetch, a file gets copied from the DFS to the local FS to a unique filename and then renamed to the target filename.

      When the filesize of the copied file doesn't match the filesize of the DFS file, the copy is performed again until the filesizes match or the maximum number of tries is reached.

      When the maximum number of tries is reached, the copy is considered unsuccessful and the temporary file gets deleted.

      The Problem is: The name of the temporary file changes with each loop, but gets deleted only after the loop. When the maximum number of tries is set to 3, this means that 2 temp files remain on the file system.

      The temporary file should be deleted inside the loop by the dfshandler if the copy was not successful.

      A follow up story – EZP-23254 – will deal with the case when filesize returned are invalid because of FS issues.

        Issue Links


          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          287d 3h 44m 1 eduardo.fernandes@ez.no 06/Aug/14 3:04 PM
          Confirmed Confirmed InputQ InputQ
          23h 24m 1 yannick.roger@ez.no 07/Aug/14 2:29 PM
          Closed Closed Reopened Reopened
          255d 22h 57m 2 eduardo.fernandes@ez.no 30/Apr/15 4:51 PM
          Reopened Reopened InputQ InputQ
          4m 34s 2 yannick.roger@ez.no 30/Apr/15 4:53 PM
          InputQ InputQ Development Development
          15s 3 yannick.roger@ez.no 30/Apr/15 4:53 PM
          Development Development Documentation Review done Documentation Review done
          4m 44s 3 yannick.roger@ez.no 30/Apr/15 4:56 PM
          Documentation Review done Documentation Review done QA QA
          6h 40m 3 miguel.jacinto@ez.no 30/Apr/15 5:58 PM
          QA QA Closed Closed
          13d 20h 57m 3 miguel.jacinto@ez.no 04/May/15 5:14 PM


            • Assignee:
              Jérôme Gamez
            • Votes:
              0 Vote for this issue
              8 Start watching this issue


              • Created: