Kleiner Patch bzgl. https und Newsletter-Versand

Sie mögen webEdition und wollen helfen, das Produkt besser zu machen? Auch Community-Mitglieder, die keine Programmierer sind, können dem Projekt helfen.
paeppi

Kleiner Patch bzgl. https und Newsletter-Versand

Beitragvon paeppi » Mo 1. Mär 2010, 14:21

Hallo miteinander,

beim Versenden von Newsletters gibt es beim aktuellen webEdition (6.0.0.8) ein kleines Problem, wenn das webEdition unter HTTPS läuft:

In /webEdition/we/include/we_modules/newsletter/weNewsletterView.php steht:

Code: Alles auswählen

			if(isset($this->settings["use_port"]) && $this->settings["use_port"]) $port = ":".$this->settings["use_port"];
			else if ($_SERVER['SERVER_PORT'] != 80) $port = ":".$_SERVER['SERVER_PORT'];
			else $port = "";
			if(isset($this->settings["use_https_refer"]) && $this->settings["use_https_refer"]) $protocol="https://";
			else $protocol="http://";
Dadurch werden, wenn die Option "Benutze HTTPS für Verweise" in den Newsletter-Optionen nicht aktiviert ist, Links mit http:// als Protokoll, und :443 als Port generiert.

Meiner Ansicht nach wäre es besser, wenn standardmäßig nicht nur der Port sondern auch das Protokoll automatisch erkannt werden. Daher hier ein möglicher Patch:

Code: Alles auswählen

			if(isset($this->settings["use_port"]) && $this->settings["use_port"]) $port = ":".$this->settings["use_port"];
			else if ($_SERVER['SERVER_PORT'] != 80) $port = ":".$_SERVER['SERVER_PORT'];
			else $port = "";
			if(isset($this->settings["use_https_refer"]) && $this->settings["use_https_refer"]) $protocol="https://";
			else if (empty($_SERVER['HTTPS']) || $_SERVER['HTTPS'] == 'off') $protocol="http://";
			else $protocol="https://";

Bei der Suche hiernach ist mir auch aufgefallen, dass die Funktion getProtocol() in /webEdition/lib/we/util/Sys/Server.php unter Umständen auch einen falschen Wert zurückgibt.

Dort steht:

Code: Alles auswählen

	public static function getProtocol()
	{
		return (empty($_SERVER['HTTPS'])) ? 'http' : 'https';
	}
Wenn PHP auf einem IIS und mit ISAPI läuft, ist $_SERVER['HTTPS'] aber nicht empty, sondern 'off'.
Daher Fix:

Code: Alles auswählen

	public static function getProtocol()
	{
		return (empty($_SERVER['HTTPS']) || $_SERVER['HTTPS'] == 'off') ? 'http' : 'https';
	}

Grüße,
paeppi

e_herrmann
Senior Member
Beiträge: 1319
Registriert: Do 22. Mai 2003, 23:25

Re: Kleiner Patch bzgl. https und Newsletter-Versand

Beitragvon e_herrmann » Mo 1. Mär 2010, 15:33

Hi,
poste das doch bitte noch einmal in die Bugbase http://qa.webedition.de/
Die Entwickler lesen hier nicht unbedingt alles mit.

Grüße Elko
Elko Herrmann
komplexx | internet.fotografie
web: http://www.komplexx.de

paeppi

Re: Kleiner Patch bzgl. https und Newsletter-Versand

Beitragvon paeppi » Mo 1. Mär 2010, 17:15

Hi,

danke für die Info, habe ich getan.

Grüße,
paeppi

MarS
Senior Member
Beiträge: 602
Registriert: Mo 2. Mai 2005, 11:30
Wohnort: Kiel
Kontaktdaten:

Re: Kleiner Patch bzgl. https und Newsletter-Versand

Beitragvon MarS » Mo 1. Mär 2010, 19:01

Das sollte für 6.0.0.9 bereits erledigt sein:
http://qa.webedition.de/tracker/view.php?id=3894

Hab' mir aber nicht im Detail angesehen, wie Armin das gelöst hat.

Gruß, Martin

paeppi

Re: Kleiner Patch bzgl. https und Newsletter-Versand

Beitragvon paeppi » Mo 1. Mär 2010, 20:20

Hab ich vorher nicht bemerkt, sorry.

Bei dem Bugreport steht als Workaround, sich über http:// in wE einzuloggen. Alternativ kann man auch einfach die Option "Benutze HTTPS für Verweise" in den Newsletter-Einstellungen aktivieren.

Grüße,
paeppi


Zurück zu „Contributions“

Wer ist online?

Mitglieder in diesem Forum: 0 Mitglieder und 14 Gäste