nextcloud / cms_pico

🗃 Integrate Pico CMS and let your users manage their own websites
https://apps.nextcloud.com/apps/cms_pico
GNU Affero General Public License v3.0
137 stars 43 forks source link

Add support for s3 external storage. #157

Closed pierreozoux closed 3 years ago

pierreozoux commented 3 years ago

Hi,

I saw the other issues. I just wanted to open one to track support for it.

Thanks for your work, and for considering this enhancement :)

PhrozenByte commented 3 years ago

Pico currently requires direct filesystem access, thus Pico CMS for Nextcloud can't support non-local storages. See #37 for more info. As soon as we have a solution for that, it doesn't really matter what non-local stroage is used, i.e. s3 external storage would work, too. This feature request is tracked in #52

pierreozoux commented 3 years ago

if it is already tracked, then I close, just the title of #52 is not really clear.

PhrozenByte commented 3 years ago

just the title of #52 is not really clear.

Yeah, that's because there are currently no plans to add filesystem abstraction to Pico, as it kinda objects the "stupidly simple" approach regarding Pico's own code. Thus it's unlikely that this will be implemented on Pico's side. However, another solution is to let Nextcloud cache external storages locally, allowing Pico to access the files locally. That's what persistent federated data basically is about. Thus it's tracked below #52. It's another solution for the same problem that is just more likely to be implemented sometime. However, this is nothing we can carry out ourselves. Also see https://github.com/nextcloud/server/issues/8459.

pierreozoux commented 3 years ago

Thanks for your kind answer :) have a nice evening :)