New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cookie detection in Internet Explorer 9 doesn't work #666

Closed
neildonkin opened this Issue Aug 24, 2012 · 1 comment

Comments

Projects
None yet
3 participants
@neildonkin

Modernizr.cookies returns true in IE9, even when cookies are disabled.

@alrra

This comment has been minimized.

Show comment
Hide comment
@alrra

alrra Aug 24, 2012

Contributor

The problem seems to be with navigator.cookieEnabled, which in IE9 is always true.
(navigator.cookieEnabled is not part of any standard so, I don't think it should be included into the test)

Contributor

alrra commented Aug 24, 2012

The problem seems to be with navigator.cookieEnabled, which in IE9 is always true.
(navigator.cookieEnabled is not part of any standard so, I don't think it should be included into the test)

@paulirish paulirish closed this in 6a0ddcd Sep 4, 2012

SlexAxton added a commit to SlexAxton/Modernizr that referenced this issue Nov 5, 2012

Merge branch 'master' of github.com:Modernizr/Modernizr
* 'master' of github.com:Modernizr/Modernizr: (81 commits)
  use setAttribute before using deprecated setAttributeNode
  explicitly set height in scrollbar test fixes false positive in IE9 (Issue #698)
  added page visibility API test
  Fix false negative of css table detect when document dir is rtl. Rewrote test. Fixes #716
  Update readme.md
  Remove BSD license and improve readme
  Added test for lossless WebP images
  Not by Addy Osmani
  binary websocket test likes to throw exceptions in Opera & Firefox and crash dolphin beta. nerfing for safety.
  navigator.cookieEnabled is a false pos in ie9. dont try it as a technique. thx @alrra. fixes #666
  bump 2.6.3pre on grunt
  bump to 2.6.3pre
  2.6.2
  When binding a function to a context, check that it actually has a bind method first.
  paul, you should always lint before you commit.
  sticky test. just formatting.
  whitespace. crlf's.
  fixing css-calc coding style, addTest gets no incoming arguments. ref #673
  Adds a test for position: (-webkit-)sticky support
  Make PhantomJS exit out of caniuse data matching early as it's unecessary
  ...

patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this issue Feb 22, 2015

patrickkettner pushed a commit to patrickkettner/Modernizr that referenced this issue Feb 22, 2015

Merge branch 'master' of github.com:Modernizr/Modernizr
* 'master' of github.com:Modernizr/Modernizr: (81 commits)
  use setAttribute before using deprecated setAttributeNode
  explicitly set height in scrollbar test fixes false positive in IE9 (Issue #698)
  added page visibility API test
  Fix false negative of css table detect when document dir is rtl. Rewrote test. Fixes #716
  Update readme.md
  Remove BSD license and improve readme
  Added test for lossless WebP images
  Not by Addy Osmani
  binary websocket test likes to throw exceptions in Opera & Firefox and crash dolphin beta. nerfing for safety.
  navigator.cookieEnabled is a false pos in ie9. dont try it as a technique. thx @alrra. fixes #666
  bump 2.6.3pre on grunt
  bump to 2.6.3pre
  2.6.2
  When binding a function to a context, check that it actually has a bind method first.
  paul, you should always lint before you commit.
  sticky test. just formatting.
  whitespace. crlf's.
  fixing css-calc coding style, addTest gets no incoming arguments. ref #673
  Adds a test for position: (-webkit-)sticky support
  Make PhantomJS exit out of caniuse data matching early as it's unecessary
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment