"index.php-less URLs" don't work with the app about appliance HOT 12 CLOSED

It does work, but you have to change a few things :)

In our documentation is written that you have to create an owncloud.conf file in your apache folder.

This is not done in the docker container. It could be done if we change the docker image. This could be discussed.

However, for now. If you create the owncloud.conf file, and then execute the occ command written in your document on how to change the URL, you will get indexless URLS.

I have created a doc site for the mean time, before we decide if we want to edit the docker image or not.

https://github.com/owncloud/docs/blob/master/modules/ROOT/pages/appliance/Index.php-less_URLs.adoc

from appliance.

Thank you very much. Just tested it and works 👍

I didn't know that the vhost is supposed to be called owncloud.conf.
The specific doc only mentions

a configured AllowOverride All directive within the vhost of your Web server is needed

The Vhost in the container is called "000-default.conf", so I thought that's the one. Didn't expect the filename to be hard-coded.

Also thanks for creating the issue, already subscribed to it.

from appliance.

No problem :)

from appliance.

Please tell me where is the difference to the current commands executed within the container.

from appliance.

current status:

if you try to follow https://doc.owncloud.org/server/10.0/admin_manual/configuration/server/index_php_less_urls.html

it does not work.

If you create owncloud.conf in apache directory as it's described in the documentation and follow

https://doc.owncloud.org/server/10.0/admin_manual/configuration/server/index_php_less_urls.html

it works.

Same commands, just one file missing.

from appliance.

But it doesn't matter how the Apache config gets named...

from appliance.

okay, but it does matter that the contents of the config are there.

You can provide your solution. :)

My solution was just the result of reading the documentation and applying it to the problem.

from appliance.

But instead of hacking such a documentation together to change a Docker image (within the appliance) that shouldn't be changed that way you should just talk to the involved people.

from appliance.

Okay, will redirect all questions regarding the docker container and it's functionality to you :)

people came to me - I delivered a solution. I am new = bad solution.

from appliance.

If I may jump to @voroyam s defense here:

Owncloud itself is already not really docker-friendly. One can't even change the login image persistently because it requires changes to the code. The entry point scripts changing config parameters to defaults at every start don't make it easier either.

I already wrote a script with the customer because owncloud just makes lots of config difficult or almost impossible to do persistently without.

In other words it's just a few lines of code more at this point.

@voroyam s doc solves the problem and he opened a new issue to fix the actual cause of that problem.

from appliance.

@voroyam I thought you know enough about the Docker images that changes like that are not persistent and that this is not really a good solution to edit files within the container manually to get some basic feature working. It's not about redirecting everything related to Docker to me or Patrick, it's about communicating about possible issues and viable workarounds.

@va1entin ownCloud itself is not entirely docker-friendly, that's a known issue. We tried to do the best to make the docker container flexible and usable enough. The entrypoint script ensures that the base configuration is really enforced by environment variables, which is totally fine related to 12factor applications. To get around the limitations of environment variables (and additional pre-installed apps) within the UCS appliance we already started to build a custom Docker image.

from appliance.

How should we proceed here?

@tboerger You suggested to modify the existing Apache config in the container so that it contains the content of the owncloud.conf @voroyam provided here
With Owncloud 10.0.10 having been released this week, it'd be great to get this into the container with the update. :)

from appliance.