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

URL Ausgabe bei Verwendung eines SSL Zertifikats fehlerhaft #229

Closed
uwe367 opened this issue Dec 18, 2017 · 14 comments
Closed

URL Ausgabe bei Verwendung eines SSL Zertifikats fehlerhaft #229

uwe367 opened this issue Dec 18, 2017 · 14 comments
Assignees
Labels

Comments

@uwe367
Copy link

uwe367 commented Dec 18, 2017

Version : phpwcms 1.9.0-rc.2 (2017/11/29, r549)
Binde ich ein SSL Zertifikat ein, z.B. eines von Let´s Encrypt, so wird bei Aufruf der Seite in der Adressleiste des Browsers folgendes ausgegeben:

https://www.neu.uwe367.de/http://www.neu.uwe367.de/
Siehe hier: http://www.neu.uwe367.de

In einer älteren Installation, wie phpwcms 1.9.0-beta.4 (2017/04/29, r545), funktioniert dies jedoch einwandfrei.
Siehe hier: http://www.test.uwe367.de/
Siehe auch im Forum:
https://forum.phpwcms.org/viewtopic.php?f=16&p=148364#p148364

@slackero
Copy link
Owner

Erstmal hat das nichts mit dem Zertifikat an sich zu tun.

Die Konfiguration an sich sollte so aussehen, wie ist Deine?

$phpwcms['site'] = '';
$phpwcms['site_ssl_mode'] = 1;
$phpwcms['site_ssl_url'] = '';
$phpwcms['site_ssl_port'] = 443;

Für mich sieht das eher nach einer falschen Konfiguration der .htaccess aus.

@uwe367
Copy link
Author

uwe367 commented Dec 18, 2017

Die .htaccess ist nicht aktiv, also auf dem Server _.htaccess und das bei BEIDEN Installationen.
Die Konfiguration der conf.inc.php sieht bei mir so aus wie du es zeigst.
Von daher weiß ich nicht was ich noch tun soll.

Bei der älteren Version klappts, bei der neuen leider nicht bzw. es wird die "normale" Url an die SSL Url angehangen..

@slackero
Copy link
Owner

Du hast Dir irgendwo eine Weiterleitung reingebaut.

@uwe367
Copy link
Author

uwe367 commented Dec 18, 2017

Es handelt sich hierbei (bei der neuen phpwcms Version) um eine neue Standartinstallation mit Standarttemplate welches sich im Originalzustand befindet und auch sonst habe ich an der Installation nichts verändert, entfernt oder hinzugefügt.

@uwe367
Copy link
Author

uwe367 commented Dec 19, 2017

Habe gerade eben noch ein paar Versuche gemacht und an der htaccess liegts definitiv nicht.
Mit aktiver htaccess und aktiviem SSL wird die http Url an die https Url angehangen und das kann nicht funktionieren.
Mit deaktivierter htaccess und aktivem SSL die Url beim Aufruf der Startseite folgender maßen angezeigt:
https://neu.uwe367.de/index.php?
Für mich sieht es aus als würde hier irgendwo etwas fehlgeleitet.

@q23
Copy link
Contributor

q23 commented Dec 19, 2017

Konnten wir hier auch reproduzieren. Manipulation an der .htaccess hat es dann beseitigt.

@uwe367
Copy link
Author

uwe367 commented Dec 20, 2017

Darf ich fragen wie deine Manipulation aussieht?

EDIT
Habs selbst bis hierher hinbekommen.
htaccess um folgende Zeilen erweitert:
RewriteCond %{SERVER_PORT} !^443$
RewriteRule (.*) https://%{HTTP_HOST}/$1 [R=301,L]

Klappt soweit schonmal. Werde weitere Versuche machen.
END EDIT

slackero added a commit that referenced this issue Dec 21, 2017
@slackero slackero self-assigned this Dec 21, 2017
@slackero slackero added the bug label Dec 21, 2017
@slackero
Copy link
Owner

slackero commented Dec 21, 2017

Bitte mal testen, ob das Problem mit der kleinen Änderung in commit 7ac51f4 final bei Euch gelöst ist.

@uwe367
Copy link
Author

uwe367 commented Dec 21, 2017

Danke für deine Mühe Oliver.
Gerade in der neuen Version getestet.
Mit aktiver .htaccess und rewrite klappts nun, mit inaktiver .htaccess (also _.htaccess) wird die Url beim Startrseitenaufruf folgendermaßen ausgegeben:
https://neu.uwe367.de/index.php?

@robinsherwood
Copy link

Mit aktiver htaccess ohne rewrite bekomme ich das gleiche .../index.php?

@slackero
Copy link
Owner

ja und, wo ist das Problem :) – ist ja nicht falsch. Habe noch einen Check integriert, der das abfangen sollte

@uwe367
Copy link
Author

uwe367 commented Dec 21, 2017

Ne.. genrell falsch ist das nicht da ja die richtige Funktion gegeben ist. Sieht halt nur komisch aus wenn beim Startseitenaufruf ein /index.php? angehangen wird ;)
Danke fürs schnelle bearbeiten :)

EDIT
Habs gerade nochmal getestet und das scheints gewwesen zu sein.
Klappt bei mir nun mit aktiver und inaktiver .htaccess sowie mit und ohne rewrite.

Nochmals DANKE fürs schnelle erledigen :)
END EDIT

@robinsherwood
Copy link

dann hole ich mir die Korrektur auch gleich - danke! :)

@q23
Copy link
Contributor

q23 commented Dec 21, 2017

danke auch von hier!

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

No branches or pull requests

4 participants