search

yakamara / redaxo_yrewrite

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

Bug - Weiterleitungen mit Query Parameters #509

Closed michael-kreatif closed 1 year ago

michael-kreatif commented 1 year ago

Hi,

glaube, das mit den Query Parameter bei Weiterleitungen ist noch nicht ganz gelöst.

Die Parameter einer Weiterleitung werden zwar gespeichert und in Zeile https://github.com/yakamara/redaxo_yrewrite/blame/main/lib/yrewrite/forward.php#L58 geprüft. Das Problem ist aber, dass im Path-Resolver bereits nur die URL ohne Query Parameter mitgegeben wird. Dadurch können die Parameter dann nicht gematched werden. https://github.com/yakamara/redaxo_yrewrite/blame/main/lib/yrewrite/path_resolver.php#L109

Beispiel:

Ruft man nun URL 2 auf, so greift bereits die Weiterleitung von URL 1, da vom Path Resolver keine Parameter übergeben werden.

( Ähnlicher Fall wie bei https://github.com/yakamara/redaxo_yrewrite/issues/508 ).

alxndr-w commented 1 year ago

Das ist doch dasselbe wie #508. Die erste matchende URL wird genommen unabhängig der Query-Parameter.

michael-kreatif commented 1 year ago

@alxndr-w nicht ganz. Beim Bug 508 geht es speziell um die Query Parameter ohne Wert. Dieser hier betrifft allgemein GET Parameter in URLs, die nicht berücksichtigt werden, ob mit Wert-Angabe oder ohne.