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

Notice: bei unbekannter Domain #390

Closed
Tracked by #5
skerbis opened this issue Dec 6, 2020 · 9 comments
Closed
Tracked by #5

Notice: bei unbekannter Domain #390

skerbis opened this issue Dec 6, 2020 · 9 comments

Comments

@skerbis
Copy link
Contributor

skerbis commented Dec 6, 2020

Warning Invalid argument supplied for foreach()
redaxo/src/addons/yrewrite/lib/yrewrite/yrewrite.php
375

Warning Invalid argument supplied for foreach()
redaxo/src/addons/yrewrite/lib/yrewrite/yrewrite.php
362

Es passiert wenn eine URL einer nicht in yrewrite verwalteten Domain aufgerufen wird.

Lösungsmöglichkeiten / Vorschläge

  • Notice in REDAXO log erstellen mit aussagekräftiger Meldung
  • Redirect auf Hauptdomain, wenn Domain nicht in YRewrite erfasst ist
  • Kann beim Hoster gelöst werden durch eine 301 Umleitung.
@skerbis skerbis changed the title Notice bei unbekannter Domain Notice: bei unbekannter Domain Dec 6, 2020
@alxndr-w
Copy link
Contributor

alxndr-w commented Dec 6, 2020

Ich empfehle die letzte Option, sie ist am zuverlässigsten und am klarsten, was passiert und warum.

@gharlan
Copy link
Member

gharlan commented Dec 6, 2020

Die Warning sollte nicht kommen, und werde ich beseitigen.
Aktuell sollte es eine 404-Antwort sein, und das würde auch so bleiben.

@skerbis
Copy link
Contributor Author

skerbis commented Dec 6, 2020

Antwort ist ein 404 , + Die Notice

193.92.XXX - - [06/Dec/2020:09:21:58 +0100] “GET /wp-login.php HTTP/1.0” 404 26815 “-” “Mozilla/5.0 (Windows NT 6.1; WOW64; rv:40.0) Gecko/20100101 Firefox/40.1

@alxndr-w
Copy link
Contributor

alxndr-w commented Dec 6, 2020

404 und dazu noch eine leere Seite vielleicht oder eine explizite Fehlermeldung? Einen 404er alleine übersehen wieder Leute, die meinen, sie hätten die Weiterleitung/Domain gerade korrekt konfiguriert. Es sollte der Admin forciert werden zu handeln und entweder eine saubere Weiterleitung beim Hosting einrichten oder die Domain in YRewrite hinterlegen.

@skerbis
Copy link
Contributor Author

skerbis commented Dec 6, 2020

Es kommt für den Besucher die Seite nicht gefunden page. Das ist soweit ok @alxndr-w . Nur die Notice im Log könnte entweder eindeutiger sein oder wegfallen.

@alxndr-w
Copy link
Contributor

alxndr-w commented Dec 6, 2020

Es gibt kein denkbares Szenario, in dem man will, dass eine zusätzliche Domain auf eine REDAXO-Installation zeigt und sie nicht von YRewrite verwaltet wird.

Deswegen sollte so eine Konfiguration zu einem gut sichtbaren Feedback für den Entwickler führen. Ein 404er-Status oder eine Info im Log ist das nicht.

Im Sinne einer guten Developer-UX würde ich mir ein klareres Feedback wünschen – bis hin zur Aufforderung.

Wer fälschlicherweise auch die Startseite als Fehlerseite eingestellt hat, wird wohl ebenfalls nichts davon merken.

@gharlan
Copy link
Member

gharlan commented Dec 6, 2020

Die Warning hier kann theoretisch auch bei einer hinterlegten Domain kommen in einem Edge Case, es war nur wahrscheinlicher, dass es über eine nicht existente Domain ging.
Auch kommt die Warning nicht immer bei Aufruf über nicht existente Domain, sondern da auch nur bei bestimmter Konstellation.
Die Warning sollte also an der Stelle auf jeden Fall abgefangen werden, und hat nichts direkt mit "Aufruf über nicht existente Domain" zu tun.

Aktuell kennt yrewrite die "default"-Domain, die für alle Bereiche greift, die außerhalb der hinterlegten Domains liegen. Und diese default-Domain, also eine nicht hinterlegte, akzeptiert jede Domain, über die die Website aufgerufen wird.
Ich bin auch schon länger am überlegen, ob man die default-Domain abschaffen sollte. Aber das ließe sich nicht in 2.x machen, ich weiß, dass die durchaus genutzt wird. Zum Beispiel muss man aktuell theoretisch gar keine Domain in yrewrite hinterlegen.

@gharlan
Copy link
Member

gharlan commented Jun 18, 2022

Die Notice müsste seit 07c1b55 schon weg sein. Oder bekommt ihr sie noch?

@gharlan gharlan closed this as completed Jun 18, 2022
@alxndr-w
Copy link
Contributor

Mir ist sie seither nicht begegnet.

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

3 participants