-
Notifications
You must be signed in to change notification settings - Fork 71
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
Why does UI5 tooling serve a directory index? #623
Comments
Looks like a duplicate of #579? |
|
We saw value in having a directory listing so we added it to ui5-server. We discussed adding an exception for index.* files but never went through with it. Then we added the But I see where you are coming from since it indeed seems to be common practice for many web servers. I think adding a We must not forget that ui5-server is a development server. We can't assume how the deployment server works. So we should not add too much magic 🪄 What do you think? Would a project configuration be helpful here or do you see a need to change the default behavior (too)? |
Personally, I don't see any value in the directory listing. I know I would change the default behavior as this will not impact bookmarks, CI or P.S. - On production: |
We are so used to getting index.html served over any other file (if present) when requesting '/'.
What is the reasoning for requiring an explicit request to index.html?
After a couple of years using UI5 Tooling, it only annoys me to have to type out the index file and I have never had any use for the directory index page.
If open to the change, this might be a good one for #506☺️
The text was updated successfully, but these errors were encountered: