-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
We want to cancel hard code index.php and modify routing. #11188
Comments
Thanks for creating the issue. @jack9603301 could you give some more information please? |
I want to arrange URL routing more scientifically and rationally, rather than having a string like |
When you use the recommended nginx (https://docs.nextcloud.com/server/13/admin_manual/installation/nginx.html) configuration index.php is removed from most visible urls. Does this help? |
I've temporarily removed the display of |
Could you be more specific what hard-coded routing you are referring to? Its pretty common to route all requests through index.php (most frameworks for php works this way). Please let me know if I misunderstood you. What problem are you trying to solve? |
The default routing for Nextcloud is sure to have
We hope that similar routing paths, such as `index.php', will not appear in the middle of the path, but more scientific and reasonable routing should be as follows:
|
https://docs.nextcloud.com/server/13/admin_manual/installation/source_installation.html#pretty-urls (you need to use the recommended nginx configuration). It does work on my local setup like you suggested. You may try the official demo https://demo.nextcloud.com (pretty-urls enabled there as well). |
How to configure on nginx. |
We manage software issue with nextcloud here. Please use https://help.nextcloud.com/ for configuration issues. You find the recommended nginx configuration here: https://docs.nextcloud.com/server/13/admin_manual/installation/nginx.html |
👍 Looks like you have configured beautiful urls for your nextcloud. Do you want me to reopen this issue? Could you provide (please edit your first post to keep this clear) more information about your use case and why the remaining links to index.php are a problem for you?
I guess you dont need this anymore? |
Part of the URL links and routing styles can be solved by adding the |
@MorrisJobke @jospoortvliet reopen? |
Usually https://docs.nextcloud.com/server/13/admin_manual/installation/source_installation.html#pretty-urls should cover all of the requested stuff. Then Nextcloud will generate only the needed URLs as wanted. Could you check that? |
Who do you want to talk to? |
Most likely you, as you have the problem. Did you followed the steps in there? Could you also check if there is a trailing slash in the override.cli.url if it only consists of the domain - something like "https://cloud.example.com/"? |
I checked the corresponding configuration records of config.php:
|
Why is there no documentation on how to do this with nginx? :/ I have |
If you use the config for Nginx that is in our documentation this should work. If not: reach out in the forums. |
Your documentation is missing the header And it worked, after I fixed the issue where it couldn't find the cgi params file. You should also add there that this link will give pretty URL, remove |
It is hoped that the hard coded index.php will change the route, so that URL routing will be more beautiful.
The text was updated successfully, but these errors were encountered: