Don't store any iitc settings in cookies

iitc-project / ingress-intel-total-conversion

ingress.com/intel total conversion user script with some new features. Should allow easier extension of the intel map.

http://iitc.jonatkins.com/

ISC License

991 stars 552 forks source link

Don't store any iitc settings in cookies #251

Closed Guria closed 11 years ago

Guria commented 11 years ago

I see cookies that IITC uses for storing settings: ingress.intelmap.layergroupdisplayed and ingress.intelmap.type It's a bad behaviour because they sent with each request to ingress servers and compromising user.

Xelio commented 11 years ago

Maybe use localStorage instead?

jonatkins commented 11 years ago

Any settings the standard map uses can remain in cookies (lat, lon, zoom), but yes, the others shouldn't be there.

We should clean out these extra cookies too