Skip to content
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

IE 8 Crashing on failed login #1

Closed
wants to merge 1 commit into from
Closed

Conversation

jslatts
Copy link

@jslatts jslatts commented Mar 2, 2011

Philip,

I have been using our django-axes project with success on one of my projects, but recently ran into an issue where users on IE8 were receiving an error:

Exception Value: Data truncated for column 'http_accept' at row 1

I traced it back to an overly large http accept header coming from the XP IE8 clients. I wasn't sure what a reasonable value for the http_accept field was, so I boosted it to 1024. Seems to work great now.

Thanks for the great work!
-Justin

@camilonova
Copy link

+1

@philipn this is a must have, we have been having this errors a lot until we figured out the browser causing the problem. Can we have this on a release?

Thanks

@codekoala
Copy link

@camilonova you might want to raise this at https://github.com/codekoala/django-axes as it's the de facto official repository for django-axes.

@aclark4life
Copy link

It looks like @jslatts is the one who should send the request to @codekoala's django-axes (https://github.com/codekoala/django-axes) if possible. Of course it's been 2 years. If it's not possible to send a new pull request then I'll merge by hand.

@camilonova
Copy link

@codekoala @philipn would be great if someone closes this project so we'll avoid confusions.

@aclark4life
Copy link

I think what's confusing is there is no "forked from codekoala/django-axes" on this repo

@camilonova
Copy link

And this is the first result from a google search

@jslatts
Copy link
Author

jslatts commented Apr 28, 2014

Closing this PR. Correct one has been merged into https://github.com/django-security/django-axes.

@jslatts jslatts closed this Apr 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants