search

YunoHost-Apps / seafile_ynh

Seafile package for YunoHost
https://seafile.com
MIT License
14 stars 19 forks source link

/seafdav is not registered in Yunohost #60

Closed roukydesbois closed 3 years ago

roukydesbois commented 4 years ago

I haven't tried myself yet, but I think that if you install 2 seafile instances, the /seafdav URLs would collapse. And if you are twisted enough to install any other app on this path, it would also cause problems. Adding a ynh_webpath_register for the seafdav path is a good start for the second issue, for the first issue I don't have a nice/user-friendly solution though.

Josue-T commented 4 years ago

Hello,

Thanks for you issue. I know that it's a problem. But it's not the only problem we have it also with /seafhttp. Actually there are not a clean way to declare multiple URL for the same app on the core. It will be fixed when https://github.com/YunoHost/yunohost/pull/861 will be merge. So for now I'm just waiting on this to implement cleanly the group-permission support on this app.

Josue-T commented 3 years ago

Should be fixed on testing branch