search

yakamara / redaxo_yrewrite

A multidomain URL rewrite engine for REDAXO
MIT License
56 stars 35 forks source link

Artikel-ID vor Weiterleitung auf Existenz prüften #498

Closed TobiasKrais closed 1 year ago

TobiasKrais commented 2 years ago

Wenn ein nicht existierender Artikel, wie z.B. https://design-to-use.de/index.php?article_id=72 aufgerufen wird, kommt eine Weiterleitungsschleife. Der Artikel sollte zuvor auf Existenz geprüft werden.

tyrant88 commented 1 year ago

Hui, ich bekomme deswegen auch gerade vermehrt von google Post, wegen Weiterleitungsschleife...

Es ist dieser neue Code-Block: https://github.com/yakamara/redaxo_yrewrite/blob/main/lib/yrewrite/yrewrite.php#L220

Wenn ich den auskommentiere, bekomme ich wieder die normale 404 Page.

fietstouring commented 1 year ago

Das ist mir auch gerade unangenehm aufgefallen!

tyrant88 commented 1 year ago

Ja, heute kam wieder Post von google.

fietstouring commented 1 year ago

ohne den code block gehen weiterleitungen aber nur ohne clang param

TobiasKrais commented 1 year ago

Ich hab gerade einen PR dazu geschrieben: https://github.com/TobiasKrais/redaxo_yrewrite/commit/96385e429aa41ffb6a0c146f9cfea6a62e22e6db. Hat jemand Zeit den zu testen?

fietstouring commented 1 year ago

Hallo, bei mir existiert die Umleitungsschleife nach wie vor, YRewrite 2.9.1

TobiasKrais commented 1 year ago

Das ist richtig. Schau dir mal das Release Datum von 2.9.1 an (August 22) und wann der Merge war (Januar 23). Der Fix kommt erst in der nächsten Version in den Installer.