search

lduarte1991 / hxat

Contains the currently-in-development project by HarvardX to bring the annotation tool currently living in the edX platform to a more accessible LTI implementation.
13 stars 7 forks source link

Fixed issues caused when you changed from wired to wireless connection #83

Closed lduarte1991 closed 7 years ago

lduarte1991 commented 7 years ago

So, I used the wrong call here originally and it caused issues with the IP matching when switching from a wired to wireless connection and back. I had to clear my cookies for things to work again. This should flush the session so that if you try to infiltrate someone's session given the obfuscated key but the IP doesn't match you don't get any of the data stored in sessions. The real user can just refresh the page (i.e. relaunch LTI tool) for things to work as intended again.

lduarte1991 commented 7 years ago

@arthurian I forgot to merge this to master when I made it. Any reason why i shouldn't?

arthurian commented 7 years ago

@lduarte1991 No reason at all. Go for it!