Problem mit dem Cache(?) im Starttemplate in eigener Anwendung

dawuid 2019-05-18 23:25:54 UTC #1

Hallo,
in meiner eigenen Anwendung finde ich die Ursache für folgendes Verhalten leider nicht:
Nach dem ersten Aufruf erfolgt in der index.php ein
Framework::redirect_link('/index.php',array('menuaction'=>'schul..
Dann rufe ich in der Anwendung den Export einer PDF-Datei auf. Wenn ich anschließend im Filter des Templates etwas auswähle, bekomme ich folgenden Fehler in den Logs:
Error reading etemplate request data for id=schul...XpneDvI... eT2 request not found / expired --> redirecting app to /egroupware/index.php
und es erfolgt ein Redirect

Dieser Fehler tritt allerdings nicht auf, wenn ich das entsprechende Template über einen Link in den Hooks aufrufe und dann den PDF-Export anstoße und anschließend im Filter die Auswahl ändere.

Kann mir jemand bitte erklären, was ich falsch mache bzw. hier nicht verstanden habe, bzw. wie ich es sogar lösen könnte? Ich hoffe ich habe es verständlich genug erklärt.

(Ubuntu 18.04, php 7.2, cache apcu)

Danke und viele Grüße, Axel

RalfBecker 2019-05-25 07:58:10 UTC #2

Zu Deiner Anwendung kann ich natürlich wenig sagen, ich kann erklären was die Meldung bedeutet:

Wenn eTemplate einen Dialog/Seite anzeigt benötigt es server-seitig State, um z.B. Information für die spätere Validierung zu speichern. Wenn das Formular dann abgesendet wird, kann es nur verarbeitet werden, wenn dieser State existiert (ist auch ein CSRF Schutz), falls nicht redirecten wir auf die gleiche URL, um eine Liste erneut anzuzeigen und damit auch den zur Verarbeitung notwendigen State wieder zu erzeugen. Danach kannst Du dann z.B. einen Filter ändern.

Da der server-seitige State natürlich Platz belegt, müssen wir vermeiden, das er beim Schließen des Fensters oder z.B. Anklicken eines Links nicht gelöscht wird. Dazu verwenden wir:
a) onBeforeClose um Server mitzuteilen, dass Fenster bzw. Formular geschlossen wurde und State gelöscht werden kann
b) beim Empfang bzw. nach der Verarbeitung wird der State gelöscht, wenn er nicht verändert wurde:

github.com

EGroupware/egroupware/blob/master/api/src/Etemplate.php#L378


			self::$validation_errors = array();
		}
		elseif (self::validation_errors(self::$request->ignore_validation))
		{
			//error_log(__METHOD__."(,".array2string($_content).') validation_errors='.array2string(self::$validation_errors));
			self::$response->generic('et2_validation_error', self::$validation_errors);
			return;
		}


		// tell request call to remove request, if it is not modified eg. by call to exec in callback
		self::$request->remove_if_not_modified();


		foreach(Hooks::process(array(
			'hook_location'   => 'etemplate2_before_process',
			'location_name'   => $template->id,
		) + self::complete_array_merge(self::$request->preserv, $validated)) as $extras)
		{
			if (!$extras) continue;


			foreach(isset($extras[0]) ? $extras : array($extras) as $extra)
			{

c) ultimativ gibt es ein Timeout, nach dem jeder State verfällt

Ich denke mal in der Art wie Du das PDF generierst kommt unsere Logik zum Schluss, dass der State nicht mehr benötigt wird und Löscht ihn. Damit schlägt das nächste Abschicken fehlt.

Dh. entweder einen Link zur Generierung des PDF verwenden, oder wenn das unpraktisch ist, eine Änderung des State erzwingen, damit dieser nicht gelöscht wird:

github.com

EGroupware/egroupware/blob/master/api/src/Etemplate/Request/Cache.php#L131


	 * As a user may open several windows with the same content/template wie generate a location-id from microtime
	 * which is used as location for request to descriminate between the different windows. This location-id
	 * is then saved as a hidden-var in the form. The above mentions session-id has nothing to do / is different
	 * from the session-id which is constant for all windows opened in one session.
	 */
	function __destruct()
	{
		if ($this->remove_if_not_modified && !$this->data_modified && isset($this->data['last_saved']))
		{
			//error_log(__METHOD__."() destroying $this->id");
			Api\Cache::unsetTree($GLOBALS['egw_info']['server']['install_id'].'_etemplate', $this->id);
		}
		elseif (($this->data_modified ||
			// if half of expiration time is over, save it anyway, to restart expiration time
			isset($this->data['last_saved']) && (time()-$this->data['last_saved']) > self::EXPIRATION/2))
		{
			//error_log(__METHOD__."() saving $this->id".($this->data_modified?'':' data NOT modified, just keeping session alife'));
			$this->cleanup();
			$this->data['last_saved'] = time();
			if (!Api\Cache::setTree($GLOBALS['egw_info']['server']['install_id'].'_etemplate', $this->id, $this->data,
				// use bigger one of our own self::EXPIRATION=4h and session lifetime (session.gc_maxlifetime) as expiration time

Ralf