filesystem changes

[elee1766]

[elee1766]

@mohammed90 can you try this?

when i run caddyfile adapt, i get

{
"apps": {
filesystems: {filesystems: [{"name":"foo","files_system":{...}}
//...

not sure if this is right still.

[elee1766]

note for myself: notify these projects

https://github.com/sagikazarmark/caddy-fs-s3 https://github.com/tecosaur/caddy-fs-git

[mohammed90]

@mohammed90 can you try this?

This works! See mohammed90/caddy-git-fs#2. I had to introduce a local interface to reach the Unwrap method, and this is alright.

when i run caddyfile adapt, i get

{
"apps": {
filesystems: {filesystems: [{"name":"foo","files_system":{...}}
//...

not sure if this is right still.

I wonder if it's possible to reduce the nesting. In other words, to make it:

{
    "apps": {
        filesystems: {
            "foo": {
                    // filesystem stuff...
                    "backend": "fs_backend_module" ...
            },
            "bar": {
                    // filesystem stuff...
                    "backend": "fs_backend_module" ...
            }
        }

or if necessary at all 🤔

[mholt]

Nit, but should it be file_systems? (Since it's two words)

[elee1766]

Nit, but should it be file_systems? (Since it's two words)

https://caddyserver.com/docs/caddyfile/options#storage

dont think it's a nit, i think you're right, you already use file_system elsewhere in the project. will change soon