minhaz

Results 171 comments of minhaz

@gyegan What kind of issues are you facing? Rather than a workaround having the library compatible with IE 11 would be best. Let's find the issue and fix it ASAP.

Assigning it to @gyegan till steps to reproduce are clear and bug is understood fully

Ok, I have a repro this library rely on the JavaScript code to attach tokens to requests in run-time. However lot of things were not supported in IE7 and thus...

Hi @gyegan I have created a workaround javascript code which worked for me in IE 11 compatibility mode. I have added it as a public gist for now: https://gist.github.com/mebjas/08e593d8e11adc5c4673ecf5be7ce018 What...

@gyegan if you not facing issues anymore, I shall start writing a generic js code. feedback needed.

Ok that is weird, line 147 is ``` js CSRFP.CSRFP_TOKEN = document.getElementById(CSRFP_FIELD_TOKEN_NAME).value; ``` so if a hidden element with id `csrfp_hidden_data_token` is not there in HTML, that means csrfp was...

@gyegan where are we on tihis?

Oh, cool I remember having tested the script. I'll perform few more tests and modify the workaround script & then work on generalizing this

I don't get how is it related to library? A general practice is to check if the class is available ``` if (ActiveXObject) { var axo = new ActiveXObject(); ......

Yes you are correct IE-11 has made changes to hide `ActiveXObject` from DOM. https://msdn.microsoft.com/library/dn423948(v=vs.85).aspx While the check here: https://github.com/mebjas/CSRF-Protector-PHP/blob/master/js/csrfprotector.js#L282 will fail in IE 11 and it will not attach the...