search

jexp / mg_client

WebClient for the MUD MorgenGrauen
http://telnet.morgengrauen.info
4 stars 1 forks source link

persist.js security issue in FF6 #13

Closed jexp closed 12 years ago

jexp commented 13 years ago

Ah, folgende Fehler hab ich da, leider wohl immer noch die mit dem persist. Error: uncaught exception: [Exception... "Security error" code: "1000" nsresult: "0x805303e8 (NS_ERROR_DOM_SECURITY_ERR)" location: "http://server.mesirii.de/~mg/js/persist/persist-min.js Line: 30"]

Error: Persist is not defined Source File: http://server.mesirii.de/~mg/js/mg_client/mg_client_persist.js Line: 1

Error: store is undefined Source File: http://server.mesirii.de/~mg/js/mg_client/mg_client_persist.js Line: 42

zesstra commented 13 years ago

Das mit dem persist-Fehler scheint was lokales in meinem profil zu sein. :-( Bei einem anderen User tritt das so nicht auf und die Seite wird scheinbar vollstaendig geladen. Jetzt muss ich nur rausfinden, wie ich mein Profil repariere. :-(

zesstra commented 12 years ago

Argl. Gnaaaaa. Weisst Du, woran es lag? An Cookies, die ich standardmaessig nicht akzeptiere.

jexp commented 12 years ago

Hmm dann entferne ich mal cookies als storage. Da passt eh nix rein.

zesstra commented 12 years ago

Cool, danke. :-)

zesstra commented 12 years ago

Hmpf. Schlechte Nachricht: es passiert immer noch, wenn man keine Cookies erlaubt. (gerade wieder fuer mg.mud.de passiert. g)

jexp commented 12 years ago

Ich stell nochmal was um, ggf. hilft es nochmal. Ich hab gelesen dass da "globalstorage" auch disabled ist, wenn cookies disabled sind. Irgendwie doof.

Ich hab das mal aktualisiert, das vorher war von 2007.

https://github.com/jeremydurham/persist-js

zesstra commented 12 years ago

Das sortierte ich auch in die wichtigeren UI-Bugs ein, weil das fuer Nutzer so aussieht, als sei die UI bzw. der Client kaputt und man keinen Hinweis hat, woran es liegt. Wenn man persist.js die Cookies nicht abgewoehnen kann, muesste man evtl. gucken, ob man Cookies speichern kann und ne Warnung ausgeben, wenn nicht...

jexp commented 12 years ago

Hmm hab mal noch ein bisschen Fehlerbehandlung zu persist.js hinzugefügt und einen "NOOP" store, der das ganze nur in einem objekt speichert. Damit sollte es jetzt eigentlich gehen, auch wenn sonst nix geht.