Skip to content
This repository has been archived by the owner on Jan 7, 2019. It is now read-only.

Slashes in Dokument-IDs #107

Open
the-infinity opened this issue Jul 23, 2013 · 5 comments
Open

Slashes in Dokument-IDs #107

the-infinity opened this issue Jul 23, 2013 · 5 comments

Comments

@the-infinity
Copy link

In einigen Kommunen kommen in den Dokument-IDs Slashes (also / ) drin vor. Das ist ein wenig unschön, da dies eigentlich reservierte Zeichen sind und die z.B. Apache aus Sicherheitsgründen automatisch rausfiltert - auch dann, wenn sie escaped sind. Das führt zu zufälllig wirkenden 404ern.

Sollte man die / durch ein anderes Zeichen (z.B. - ) ersetzen (und damit dann die Zuordnung zu Orginal-Dokumenten schwerer machen) oder die URLs so lassen und Apache und Co anweisen die Slashes anzunehmen? Im Apache Support Channel habe ich eine sehr warme Empfehlung zum Beseitigen der Slashes bekommen.

@marians
Copy link
Owner

marians commented Jul 23, 2013

Bislang habe ich Slashes auch immer durch Dashes (also Minuszeichen) ersetzt. Habe ich das in der neuen Version abgeschafft? Vielleicht sollten wir es wieder aufnehmen.

@the-infinity
Copy link
Author

Yep, Du wolltest sogar ein Redirect dazu schreiben: #101 - vielleicht doch lieber wieder ersetzen lassen?

@marians
Copy link
Owner

marians commented Aug 3, 2013

Letztlich wollen mir Slashes in der ID auch nicht wirklich gefallen.

@the-infinity
Copy link
Author

Hmm. Der Ansatz von damals klappt nur bei einigen Kommunen nicht, weil das - zeitweise auch verwendet wird. Beispiel: https://www.duisburg.de/ratsinformationssystem/bi/vo0050.php?__kvonr=20056078 - da geht ein einfaches real_reference = reference.strip().replace('-', '/') in https://github.com/marians/offeneskoeln/blob/v1.0/webapp/dispatch.py schief. Werde also ein URL Feld in der Datenbank einführen (mit Möglichkeiten zum Alias, da kann man später dann auch sprechende URLs generieren und Weiterleitungen wie die in #101 angesprochene realisieren).

@the-infinity
Copy link
Author

So. Ist nun wie beschrieben mit zusätzlichem URL Array-Datenbankfeld im OpenRuhr Repo drin: https://github.com/OpenRuhr/ris-web/commits/master

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants