idee: nicht existierende jahre auf datei umbiegen #30

Open
vv01f wants to merge 3 commits from c3d2/nix-config:ds-future into master
vv01f commented 2 months ago
Collaborator

Tut das oder wie muss das anders?

Tut das oder wie muss das anders?
vv01f added 1 commit 2 months ago
Owner

Das gilt für alle 404? Können wir das iwie nach ^/\d{4}/$ filtern?

Das gilt für alle 404? Können wir das iwie nach `^/\d{4}/$` filtern?
Poster
Collaborator

Bin total dafür wenn das geht :)

Bin total dafür wenn das geht :)
vv01f added 1 commit 2 months ago
vv01f added 1 commit 2 months ago
Owner

Die Regex endet auf /.*$, matcht also auch bspw. das nichtexistierende /2019/favicon.ico und erzeugt dort einen Redirect. So gewollt?

Die Regex endet auf `/.*$`, matcht also auch bspw. das nichtexistierende `/2019/favicon.ico` und erzeugt dort einen Redirect. So gewollt?
Poster
Collaborator

Die Regex endet auf /.*$, matcht also auch nichtexistierende $Dateien und erzeugt dort einen Redirect. So gewollt?

Gewollt ist ein Redirekt für alle Jahre ab dem aktuellen Kalenderjahr, auch bei beliebigen Dateien darunter… aber ich kenne keine Möglichkeit das aktuelle Datum in RegEx einzubeziehen und daher ist das IMO ein verschmerzbarer Fall. Sowas ließe sich über den Inhalt des Redirect Zieles abfangen.
Oder kennst du eine gute Möglichkeit, die nicht Weiterleitungen auch bei der nicht existierenden /2099/fahrplan.html usw. ausführt?

> Die Regex endet auf `/.*$`, matcht also auch nichtexistierende $Dateien und erzeugt dort einen Redirect. So gewollt? Gewollt ist ein Redirekt für alle Jahre ab dem aktuellen Kalenderjahr, auch bei beliebigen Dateien darunter… aber ich kenne keine Möglichkeit das aktuelle Datum in RegEx einzubeziehen und daher ist das IMO ein verschmerzbarer Fall. Sowas ließe sich über den Inhalt des Redirect Zieles abfangen. Oder kennst du eine gute Möglichkeit, die nicht Weiterleitungen auch bei der nicht existierenden `/2099/fahrplan.html` usw. ausführt?
This pull request can be merged automatically.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.