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

Image Caching #45

Closed
darwin26 opened this issue Jan 24, 2014 · 22 comments
Closed

Image Caching #45

darwin26 opened this issue Jan 24, 2014 · 22 comments

Comments

@darwin26
Copy link

Hi, bin mir nicht sicher ob das n Community bug ist oder am Server liegt. Sobald man jedoch "nur" das AddOn community installiert und aktiviert hat, funktioniert das Caching der Bilder nicht mehr und wirft bei jedem laden ein 200 raus statt einem 304 (not modified).

Ich habe nun testweise in der /addons/community/config.inc.php unterhalb des session_start() (ca. zeile 14)
das hier eingefügt:

header('Cache-Control: must-revalidate, proxy-revalidate, private');

Dann funktioniert s auch wieder mit dem Image Caching. :)

Redaxo 4.5.1 + community

grz. Chris

@ghost
Copy link

ghost commented Jan 24, 2014

Die Zeile header('Cache-Control: must-revalidate, proxy-revalidate, private'); ist für die Image Manager Bilder rausgeflogen aus R4.5.1, da sie ein sauberes setzen der Caching Einstellungen über die .htaccess Datei verhindert hat. Siehe redaxo/redaxo4#229

LIegt es vielleicht da ran? Ist es ohne Community auch so?

Btw: Caching sollte man über die .htaccess Datei einstellen, so wie hier: https://github.com/RexDude/resource_includer/blob/master/install/_htaccess#L39-L63

@darwin26
Copy link
Author

Hi. Ohne Community verhält es sich richtig. Ich hab es mit der Demo getestet inkl. Seo42 ... Und der htaccess von rexdude. Sobald Community aktiviert und installiert ist, wird ne 200 geliefert...

Gruß Christian Kolloch
Gesendet mit meinem HTC

----- Reply message -----
Von: "RexDude" notifications@github.com
An: "dergel/redaxo4_community" redaxo4_community@noreply.github.com
Cc: "Chris" info@darwin-design.de
Betreff: [redaxo4_community] Image Caching (#45)
Datum: Fr., Jan 24, 2014 17:08

Die Zeile header('Cache-Control: must-revalidate, proxy-revalidate, private'); ist für die Image Manager Bilder rausgeflogen aus R4.5.1, da sie ein sauberes setzen der Caching Einstellungen über die .htaccess Datei verhindert hat. Siehe redaxo/redaxo4#229

LIegt es vielleicht da ran? Ist es ohne Community auch so?

Btw: Caching sollte man über die .htaccess Datei einstellen, so wie hier: https://github.com/RexDude/resource_includer/blob/master/install/_htaccess_no_seo42#L39-L63


Reply to this email directly or view it on GitHub.

@ghost
Copy link

ghost commented Jan 24, 2014

Seltsam. Hats vielleicht was mit dem auth_media Plugin zu tun? Probier mal die Zeile zu entfernen: https://github.com/dergel/redaxo4_community/blob/master/plugins/auth_media/classes/class.rex_com_auth_media.inc.php#L34

@darwin26
Copy link
Author

HI,
ich hab jetzt (lokal) mal alles durchgespielt. Mal mit Plugins, mal ohne, auskommentierter header (auth_media) ... Es scheint als wäre das Problem am AddOn community selbst... habs mal durchgeforstet, seh aber sleber kein Problem. Könnte es auch iwie im zusammenhang mit dem ImageManager stehen?

@staabm
Copy link
Collaborator

staabm commented Jan 31, 2014

kannst du alle addons auflisten die du hier verwendest?

@darwin26
Copy link
Author

Redaxo 451 demo
installierte / aktive Addons:

  • be_style (+ agk_skin
  • community (ohne plungins installiert)
  • image_manger
  • metainfo
  • ressource_includer
  • seo42
  • textile
  • xform (+ email, manager + setup)

@darwin26
Copy link
Author

alles andere hab ich deinstalliert

@darwin26
Copy link
Author

sobald ich das AddOn "community" de-installiere... gehts ..

@ghost
Copy link

ghost commented Jan 31, 2014

Und hast du auch schon die anderen Addons abgeknipst? Und die .htaccess entfernt oder zurückgesetzt?

@darwin26
Copy link
Author

HI,
also nochmal mit einer reinen 451-Demo ;)...
Lediglich die standards sind drin + community ... sobald community on = 200er... off = 304 ...

Ich habe ein testbild auf der Startseite via Modul platziert:
Modul-Ausgabe:
img src = index.php?rex_img_type=rex_mediapool_detail&rex_img_file=img_9589.jpg

Addons / installiert und aktiviert

  • community (ohne plungins installiert)
  • image_manger
  • metainfo
  • phpmailer
  • xform (+manager)

Teste lokal auf OS X / MAMP / php 5.4.4 /

grz. chris

@ghost
Copy link

ghost commented Jan 31, 2014

Ich kanns reproduzieren :) Im Cache-Control Header des IM Bildes stehen wilde Sache drin:
Cache-Control: no-store, no-cache, must-revalidate, post-check=0, pre-check=0

@ghost
Copy link

ghost commented Jan 31, 2014

http://stackoverflow.com/a/681584

session_cache_limiter('none'); in der Community config.inc.php (an oberster Stelle) scheint das Prob zu beheben :) Chris?

@darwin26
Copy link
Author

Jo,
platziert in der config.inc.php (community AddOn) >> alles gut (wirft nun 304 ;))

grz. Chris

@ghost
Copy link

ghost commented Jan 31, 2014

@staabm
@dergel
Sollte direkt rein wenns keine Einwände gibt, oder?

@ghost
Copy link

ghost commented Jan 31, 2014

Oder gehörts eher in den Core, weil ja sobald ein anderes Addon etc. ne Session startet dieses Verhalten vermutlich auftreten wird. Hier ist es z.B. ja schon drin, nur die IM Bilder gehen hier nicht drüber: https://github.com/redaxo/redaxo4/blob/master/redaxo/include/functions/function_rex_client_cache.inc.php#L84

@staabm
Copy link
Collaborator

staabm commented Jan 31, 2014

ich meine wir hatten eine ähnliche diskussion schon in R5,... kann das sein?

@ghost
Copy link

ghost commented Jan 31, 2014

Zu R5 kann ich nix sagen. Aber ich habe gerade herausgefunden, dass dieser Commit jetzt mit der 4.5.1 das Prob verursacht, dank mir ;)
redaxo/redaxo4@a3f48b4

Habe eben session_cache_limiter('none'); wieder zurückgeschoben in die rex_send_content, aber alleine stehend (ohne die Cache-Control Zeile) hat es komischerweise keine Wirkung.

Ich glaube redaxo/redaxo4#229 muss nochmal eröffnet werden, oder?

@gharlan
Copy link
Member

gharlan commented Jan 31, 2014

session_cache_limiter ist dort zu spät, der Aufruf muss kommen bevor die Session gestartet wird. In R5 hab ich es ganz vorn eingefügt: https://github.com/redaxo/redaxo/blob/master/redaxo/src/core/boot.php#L38
Allerdings übergebe ich dort false statt 'none'.

@ghost
Copy link

ghost commented Jan 31, 2014

Sollte man für R4 dann auch so machen denke ich. Ob false oder none einen Unterschied macht, weiss ich net. Konnte dazu nix finden.

@ghost
Copy link

ghost commented Feb 14, 2014

Das ganze wurde jetzt für die REDAXO 4.6 gefixt.

@dergel
Copy link
Member

dergel commented Sep 3, 2014

ich werde gerade nicht ganz schlau und bevor ich in den tiefen rumsuche und wühle .. ist das problem behoben ? kann ich das issue schliessen ?

@ghost
Copy link

ghost commented Sep 5, 2014

Ja kann geschlossen werden. Habs grad nochmal getestet mit R4.6.1. Wird jetzt alles sauber cachet :)

@gharlan gharlan closed this as completed Sep 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants