Js file name is wrong and improvements to js logic is required

Adding patch for the fix.

Thank you @sandipta !
This patch is correct.

With that said , I'm still learning/evaluating how this module works (or not).

Ok patch #2 doesn't do the job, but I do have a new patch that does.

patch 5 works

New patch, this covers all normal use cases, only kills session when the browser is closed

Another option would be to use the visibilitychange event which works more reliably on mobile platforms than beforeunload.

See my comments #9. This isn't perfect so we're likely going to uninstall this module. With that said, by all means continue working on this. The security constraints of modern browsers hasn't yet allowed an api connect between the actual close button and a js event that directly corresponds to it. The js unload event is too broad. It was a nice idea in theory but in practice just doesn't work as we had hoped. With that said, this idea could be useful for other use cases that I'm currently not working on.