Bug - Weiterleitungen mit Query Parameters ohne Wert liefern falsches Ergebnis

yakamara / redaxo_yrewrite

A multidomain URL rewrite engine for REDAXO

MIT License

56 stars 36 forks source link

Bug - Weiterleitungen mit Query Parameters ohne Wert liefern falsches Ergebnis #508

Closed michael-kreatif closed 1 year ago

michael-kreatif commented 2 years ago

Hey Leute,

angenommen, es gibt folgende Weiterleitungen:

URL 1: produkt.html?abc -> Seite A
URL 2: produkt.html -> Seite B
URL 3: produkt.html?def -> Seite C

URL 3 wird aufgerufen. Dann greift bereits URL 1 und man wird auf Seite A weitergeleitet. Folgende Zeile bewirkt, dass zwar auf die GET Parameter geprüft wird. Wenn diese aber leer sind, dann wird die URL 1 nicht übersprungen, weil der Get Parameter keinen Wert hat.

https://github.com/yakamara/redaxo_yrewrite/blame/main/lib/yrewrite/forward.php#L58

alxndr-w commented 2 years ago

Zum besseren Verständnis des Anwendungsfalls: In welchem Szenario will man get-Parameter ohne Wert?

michael-kreatif commented 1 year ago

@alxndr-w gute Frage, wusste ich zuerst auch nicht, dass das überhaupt RFC valide ist. Das konkrete Projekt hat jemand anders realisiert bzw. leider gibt es die URLs mit get-Parameter ohne Wert bereits, die weitergeleitet werden sollten.

Meines Erachtens ist es aber eher ein Bug mit niedriger Priorität, weil wie du schon richtig sagst, sehe ich auch keine guten Praxisfälle, wo es Sinn macht, get-Parameter ohne Wert einzusetzen. Daher wollte ich diesen Bug auch getrennt dokumentieren ;).