Skip to content
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

MediaManager: Bilder werden immer mit Status 200 geliefert. #1031

Merged
merged 1 commit into from
Feb 12, 2017

Conversation

akrys
Copy link
Contributor

@akrys akrys commented Feb 12, 2017

Wollte mir eigentlich mal ansehen, wie man den Expires-Header setzt, weil wir kürzlich viele Probleme mit generierten Bildern haben. Bin in der 5.3-Beta1 dann darauf gestoßen, dass die Bilder immer generiert und mit Status 200 ausgeliefert werden.
Beim genaueren untersuchen ist mir aufgefallen, dass es die Varable $header in der Funktion rex_media_manager::sendMedia gar nicht gibt.

Das dürfte das Problem aus #1025 sehr wahrscheinlich lösen.

…he-Control auf "cache" steht.

Wahrscheinlich ist es genau das beschriebene Problem in redaxo#1025
@hirbod
Copy link
Contributor

hirbod commented Feb 12, 2017

Dürfte vermutlich meine Probleme lösen, die ich #1025 gepostet habe
Edit: Sehe gerade, du hast das schon erwähnt, sry!

@hirbod
Copy link
Contributor

hirbod commented Feb 12, 2017

edit: doch, passt, hab tomaten auf den augen

@akrys
Copy link
Contributor Author

akrys commented Feb 12, 2017

Problematisch ist, dass die Variable $header verwendet wird, die in der Funktion aber nicht da ist. Dadurch wird die Funktion rex_response::sendLastModified gar nicht aufgerufen und das Bild wird immer neu generiert.

@hirbod
Copy link
Contributor

hirbod commented Feb 12, 2017

Ja, hab mich korrigiert. Ich hab schneller geschrieben als mein Hirn deinen fix verarbeitet hatte.

@akrys
Copy link
Contributor Author

akrys commented Feb 12, 2017

Macht nix. An einem Sonntag Nachtmittag darf das passieren ;-)

@hirbod
Copy link
Contributor

hirbod commented Feb 12, 2017

@gharlan der Fix behebt zwar bei einigen Bildern das Problem, aber manche werden immer noch 5 Sekunden lang ausgeliefert. Irgendwo ist noch ein Bug in dem Betabuild des Managers. Auch bekommen manche User dieses ENT:: Problem

bildschirmfoto 2017-02-12 um 15 01 39

@gharlan gharlan added the Bug label Feb 12, 2017
@gharlan gharlan added this to the REDAXO 5.3 milestone Feb 12, 2017
@gharlan gharlan merged commit deceb77 into redaxo:master Feb 12, 2017
@gharlan
Copy link
Member

gharlan commented Feb 12, 2017

Danke!

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

Successfully merging this pull request may close these issues.

None yet

3 participants