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

Der Benutzer soll auf die gewünschte Seite weitergeleitet werden #32

Open
mojoaxel opened this issue Aug 16, 2013 · 10 comments
Open

Der Benutzer soll auf die gewünschte Seite weitergeleitet werden #32

mojoaxel opened this issue Aug 16, 2013 · 10 comments
Labels

Comments

@mojoaxel
Copy link
Contributor

Der Benutzer wird durch die Landingpage beim Surfen unterbrochen. Wenn der Benutzer auf "Weitersurfen" klickt soll er automatisch zu der vorher gewünschten Seite weitergeleitet werden.

Eigentlich sollte das bereits funktionieren (siehe #24) aber es gibt im Realbetrieb noch Probleme.

@StilgarBF
Copy link
Contributor

ich vermute, dass das history.back, dass ja verwendet wird bei 302ern nicht in jedem Browser gleich reagiert(bzw umgesetzt ist)
siehe http://en.wikipedia.org/wiki/List_of_HTTP_status_codes#3xx_Redirection
Der Server teilt ja mit "Moved Temporarily" - uU geht der Browser dann beim "back" davon aus, dass er ohnehin wieder umgeleitet wird. (pure Spekulation)

Laut wikipedia

"However, some Web applications and frameworks use the 302 status code as if it were the 303."
die 303:
"The response to the request can be found under another URI")

Unter Umständen wäre die 307 einen Test wert:

"Temporary Redirect (since HTTP/1.1)" ... "... future requests should still use the original URI."

Ich würde empfehlen mit i-Geräten nochmal intensiv zu testen, ob der Fehler wie in der ML beschrieben reproduzierbar ist.
Wenn ja, 307 testen.

Mir stehen iPhone - iPad etc nicht zur Verfügung.

@mojoaxel
Copy link
Contributor Author

Ich hab mit commit f5b2586 mal versucht das Problem zu beheben. Folgende zwei Dinge habe ich probiert:

  • Eine 2 Sekunden Verzögerung soll dem Squid Proxy mehr Zeit geben den Eintrag in die Datenbank zu schreiben (siehe Mail von Casandro)
  • Ich hab den Aufruf von history.go(-1) auf history.back() weil ich gelesen hatte, dass es wohl Probleme auf manchen Browsern geben kann.

Wenn es weiterhin Probleme gibt bitte einfach Issue sofort wieder öffnen...

@mojoaxel
Copy link
Contributor Author

Es scheint immer noch Probleme mit der Weiterleitung auf dem iPhone zu geben :-)

@mojoaxel mojoaxel reopened this Aug 17, 2013
mojoaxel added a commit that referenced this issue Aug 17, 2013
@sebastian-schmid
Copy link
Contributor

Auf dem iPhone kommen die drei blinkenden Vierecke und nichts weiter passiert. Kann mir nach wie vor nicht erklären warum...

@mojoaxel
Copy link
Contributor Author

@sebastian-schmid Seltsam! Ich hab leider auch grade kein iPhoine zur Hand. Du könntest ja zum test mal einen anderen Browser auf dem iPhone testen. Wenns damit klappt würde ich mir den iPhone Browser im Simulator mal genauer zur Brust nehmen.

@sebastian-schmid
Copy link
Contributor

Mit Opera auf dem iPhone scheint es zu funktionieren.

Mit Firefox auf dem Mac hab ich allerdings, wie in Safari am Mac, das Problem das der klick auf weitersurfen funktioniert. Ich aber einfach ein leeres Browserfenster bekomme (also so als ob der Browser eben erst gestartet wurde). Sprich die Weiterleitung auch nicht wirklich läuft.

@FreifunkUFO
Copy link

wäre es nicht sinnvoller, dass sich der splash-webserver die URL adresse merkt, die ursprünglich aufgerufen wurde? das muesste dann noch durch den splash-prozess weitergegeben werden (get oder post) und auf den "weitersurfen" button übertragen..

@mojoaxel
Copy link
Contributor Author

Casandro hat die Landingpage erstmal offline genommen. Jetzt wird es nur schwer den Fehler zu finden...

@sebastian-schmid
Copy link
Contributor

Finde die Idee von FreifunkUFO gut.
Ist es generell Möglich die URL abzugreifen und an per Skript dann die Splashpage mit passendem Link zu generieren?

@FreifunkUFO
Copy link

na die URL steht im http-header/umgebungsvariablen, zumindest das erste mal, wenn der webserver die umleitung "abkriegt".. da ist es ja quasi "catchall"...

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

No branches or pull requests

4 participants