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

Unable to set file system permissions correctly despite Flysystem update

    Details

      Description

      The FlysyemBundle update implemented in EZP-25676 was not enough to to bring file permissions control to the user since there are no implemented settings.

      To fully mimic the the legacy dfs handler, some those settings must be implemented.

      1. dev.log
        451 kB
        Nuno Oliveira

        Issue Links

          Activity

          Hide
          Yannick Roger (Inactive) added a comment -

          This issue doesn't have any steps to reproduce. Please provide some in order to reproduce the bug.

          Show
          Yannick Roger (Inactive) added a comment - This issue doesn't have any steps to reproduce. Please provide some in order to reproduce the bug.
          Hide
          Bertrand Dunogier added a comment -

          Unless I am missing something, it is a fairly simple change, [~yannick.roger@ez.no].

          The constructor of the default filesystem adapter accepts a 4th argument that sets files & directory permissions. It is currently not supported by our service definition.

          We could provide a workaround based on a compiler pass, and then consider adding it to the kernel, as a convenience parameter.

          Show
          Bertrand Dunogier added a comment - Unless I am missing something, it is a fairly simple change, [~yannick.roger@ez.no] . The constructor of the default filesystem adapter accepts a 4th argument that sets files & directory permissions. It is currently not supported by our service definition. We could provide a workaround based on a compiler pass, and then consider adding it to the kernel, as a convenience parameter.
          Hide
          Bertrand Dunogier added a comment -

          Actually, after looking into it, there are settings for this...

          ezpublish:
              system:
                  default:
                      io:
                          # Permissions applied by the Local flysystem adapter when creating content files and directories in storage.
                          permissions:
                              files:                '0644'
                              directories:          '0755'
          

          [~eduardo.fernandes@ez.no] could you check if it matches what is requested ?

          Show
          Bertrand Dunogier added a comment - Actually, after looking into it, there are settings for this... ezpublish: system: default: io: # Permissions applied by the Local flysystem adapter when creating content files and directories in storage. permissions: files: '0644' directories: '0755' [~eduardo.fernandes@ez.no] could you check if it matches what is requested ?
          Hide
          Bertrand Dunogier added a comment -

          Sending back to InputQ as feedback has been provided.

          Show
          Bertrand Dunogier added a comment - Sending back to InputQ as feedback has been provided.
          Hide
          André Rømcke added a comment - - edited

          Added additional documentation to make it clear configuration should be non string value as needs to be parsed into an octal number:
          https://doc.ez.no/display/DEVELOPER/Asset+Management#AssetManagement-Configurethepermissionsofgeneratedfiles

          Show
          André Rømcke added a comment - - edited Added additional documentation to make it clear configuration should be non string value as needs to be parsed into an octal number: https://doc.ez.no/display/DEVELOPER/Asset+Management#AssetManagement-Configurethepermissionsofgeneratedfiles
          Hide
          Bertrand Dunogier added a comment -

          Sending to QA so that the current doc can be tested.

          [~eduardo.fernandes@ez.no] / [~nuno.oliveira@ez.no]: Please make sure that the status and answer are forwarded to the customer, and move the issue forward if nothing else is needed from 3rd line.

          Show
          Bertrand Dunogier added a comment - Sending to QA so that the current doc can be tested. [~eduardo.fernandes@ez.no] / [~nuno.oliveira@ez.no] : Please make sure that the status and answer are forwarded to the customer, and move the issue forward if nothing else is needed from 3rd line.
          Hide
          Rui Silva (Inactive) added a comment -

          Documentation validated by QA, by testing it on a 1.9.

          Show
          Rui Silva (Inactive) added a comment - Documentation validated by QA, by testing it on a 1.9.

            People

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

              Dates

              • Created:
                Updated:
                Resolved: