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

Swagger UI not working in IE 8 - Unable to get property 'content-type' of undefined or null reference #407

Closed
svalluri opened this issue Feb 27, 2014 · 3 comments

Comments

@svalluri
Copy link

Migrated from 1.2 to 2.0.21 hoping to resolve IE 8 issues related to console.log and defineProperties (shred.bundle.js). Now getting a different error - Unable to get property 'content-type' of undefined or null reference.

swagger.js, Line: 1119, Column: 5

@fehguy
Copy link
Contributor

fehguy commented Feb 28, 2014

Hi, please pull from master, there was a bug in swagger-js that was expecting the Array.indexOf function to work.

But I will note that, IE8 is not 100% supported. There are CORS issues so if your ui is not running on the same machine as the api you may get access-denied errors. It can be done, but there needs to be a pretty good argument for adding a bunch of baggage into swagger-js to detect IE8 in swagger-js and add all the work-arounds, as explained here:

http://blogs.msdn.com/b/ieinternals/archive/2010/05/13/xdomainrequest-restrictions-limitations-and-workarounds.aspx

@fehguy fehguy closed this as completed Feb 28, 2014
@svalluri
Copy link
Author

Thanks Tony!

Pulled the latest from master. It’s working in IE 8, after commenting out the log function, As getting error at Swagger-ui.js line 66, log.history = log.history || [];

Then got “Unable to get property ‘call’ of undefined or null reference
File: highlight.7.3.pack.js, Line: 1, Column: 274.

Thanks again for the quick response.

@fehguy
Copy link
Contributor

fehguy commented Mar 19, 2014

Hi you shouldn't need to make any changes, such as commenting out the log function. Can you please do a check with master again?

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

No branches or pull requests

2 participants