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

DFS - tmp files left if the file size is not identical to the original one

    Details

      Description

      EZP-21825 fix some of the temporary files left in the file system if there's a failure in the copy/rename process between the local and the NFS file systems.

      But, if after the copy the file size of the original and the copied file are different, the tempo files will be left undeleted.

      Steps to Reproduce

      This is a hard issue to reproduce since it will only happen if there are any NFS errors, but it's possible to achieve the same reproduce.
      Edit

      + if( $loopCount != $this->maxCopyTries )
      +{
      +   $dfsFileSize += 10;
      +   $localFileSize -= 10;  
      +}
      $localFileSize = filesize( $filePath );
      if ( $dfsFileSize == $localFileSize )
      {
      	return $filePath;
      }
      

        Issue Links

          Activity

          Show
          Yannick Roger (Inactive) added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1059
          Show
          Yannick Roger (Inactive) added a comment - - edited Fixed in master : https://github.com/ezsystems/ezpublish-legacy/commit/bac700aa148188b77d6b02dc4212b96daa990e3a
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Tested on master.
          tc-2232

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved Tested on master. tc-2232
          Hide
          Paulo Nunes (Inactive) added a comment -

          QA Approved

          Globally tested on master, 5.3.3 and in 5.2

          Show
          Paulo Nunes (Inactive) added a comment - QA Approved Globally tested on master, 5.3.3 and in 5.2
          Hide
          Eduardo Fernandes (Inactive) added a comment -

          Reopening for backport to eZP 5.1

          Show
          Eduardo Fernandes (Inactive) added a comment - Reopening for backport to eZP 5.1
          Hide
          Miguel das Neves Jacinto (Inactive) added a comment -

          QA approved on 5.1

          Show
          Miguel das Neves Jacinto (Inactive) added a comment - QA approved on 5.1

            People

            • Assignee:
              Unassigned
              Reporter:
              Eduardo Fernandes (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: