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

Fix EZP-24312: Default installation content results in 404 errors #240

Closed
wants to merge 1 commit into
base: master
from

Conversation

3 participants
@brookinsconsulting
Copy link
Contributor

brookinsconsulting commented Apr 27, 2015

Hello,

This issue and PR are a direct result of testing the latest / recent ezpublish-community repository reverts / changes for default installation errors and any other problems which might be blockers for the creation of the next ezpublish-community eZ Publish 5 community build, currently in preparation phase.

The default installation of ezpublish-community using demobundle and ezdemo ezpackage content which provides selected video content results in 404 errors on default index page.

This is a temporary problem for both eZ Publish 5.x and eZ Platform.

This is direct result of the ezpublish-kernel not yet providing a new stack (Symfony) based replacement for the content/download module view which is required to abstract the serving of binary file content object content.

The problem is that the demobundle and ezdemo extension provided default content during a default installation make use of binary file content which requires these mod_rewrite rules be in your virtual host configuration ... or a default installation described will generate 404 errors when loading the index page of the default user siteaccess.

The 404 errors can be solved today by editing your virtual host configuration mod_rewrite rules and adding the new rules provided by this PR, commented out by default.

This PR provides for the education to developers on this important issue which has for some time now generated page load 404 errors (and large amounts of confusion to developers and users) while retaining the maximum security in a default installation.

This issue has been uniquely documented here: https://jira.ez.no/browse/EZP-24312

Please let us know your thoughts.

Cheers,
Brookins Consulting

@brookinsconsulting brookinsconsulting force-pushed the brookinsconsulting:ezp_ezdemo_required_rewrite_rules_documentation branch from 66150a7 to 1e4d8cb Apr 27, 2015

@andrerom

This comment has been minimized.

Copy link
Member

andrerom commented Apr 28, 2015

I think @bdunogier is working on a fix for this, so depending on his progress we should try to avoid to many people applying workarounds that makes the original files public.

@andrerom

This comment has been minimized.

Copy link
Member

andrerom commented Jun 3, 2015

re ping @bdunogier

@crevillo

This comment has been minimized.

Copy link
Contributor

crevillo commented Jun 3, 2015

@brookinsconsulting the 404 error you was getting was coming from the video showed in the bottom right part of the demo homepage? if so, ezsystems/DemoBundle#157 should solve it.
For the other files, the content/download work from @bdunogier should fix those too.

imho this should be closed and not add more noise to the vhost doc.

@brookinsconsulting

This comment has been minimized.

Copy link
Contributor Author

brookinsconsulting commented Jun 3, 2015

@crevillo

Yes. Agreed.

The problem is now solved the right way and this PR is not needed (and a poor way to address the actual problem). Hooray for the new content/download support in the new stack kernel!

We will now close this PR. Thank you all for your excellent support!

Cheers,
Brookins Consulting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment