-
-
Notifications
You must be signed in to change notification settings - Fork 320
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
Not working for ios 11 #51
Comments
Also having this issue :( |
Hi guys, can you please provide more info?
|
Apologies, I was on Cordova 7.0.1, cordova-ios 4.5.3, I can't remember the others though. I wasn't specifying a serializer, does that make it default to urlencoded? I have since re-setup my project and reinstalled the plugins and node modules and everything is working correctly. I've been using it with Ionic, and it's possible that it was just an issue with how I was using the plugin, rather than a genuine issue. The API server I am working with can fail with a 404 on the options request if data is passed incorrectly (not the best response code I know). So I think blindly updating plugins without checking changes and not consulting your docs has been the issue with the project I'm working on. TLDR: Reinstalling the plugin and node modules has fixed the issue. I have been unable to replicate since. |
cli-7.0.1 , ios: 4.4.0 , cordova-plugin-advanced-http version: 1.9.0, adobe phonegap build site, |
@ganey thanks for your feedback! @havenht can you show us some example code and result (error message)? You can try to post some data to http://httpbin.org/post for example. |
i use your basic examples for post and get. Both doesn't work for ios 11, but work for ios 10 and below. You should test it out on ios 11 on your end. cordova.plugin.http.get('https://google.com/', { |
Actually, I did already test on iOS 11 (Simulator and Device) and I don't face any problems. Atm, my specs are configured to run against iOS 11 locally on my dev machine (see here) and to run against iOS 10.3 on saucelabs (see here). You can have a look into this file to see the specs I've implemented. Here you can see the result logs of my commits built on Travis and tested on SauceLabs. |
I am getting the http 200 response in the error handler as well. I'm using the ionic native wrapper if that matters, and i'm only on iOS 9.3.5, so I don't think it's an ios 11 issue |
Hi nikmartin, is your backend a public webservice I can reach and test against? Did you already try to post some data to http://httpbin.org/post? I'd like to find out if it's caused by a specific value in the response, because I don't face any of the mentioned problems on my side. |
Yes, it's a shared iCal file located here: https://outlook.office365.com/owa/calendar/f601ad847ba74f0b87dc64989512bd4d@mcpss.com/46c96d0559e749a796af4aca6623c02d15003821461964994350/calendar.ics I fetch it using ( this is angular/typescript using the http native ionic plugin wrapper):
And the catch receives a 200 status and the message is the actual output |
I will try this now and let you know. BTW the plugin works fine with that URL on Android, this is only on iOS, but occurs on every version I've tried, from 9.3 to current. |
POST-ing from the app to httpbin/org/post using the plugin on iOS works. Here's the response when I run the function above, with the iCal url:
|
I temporarily am working around the issue by just checking the errror status code and if it's 200 assume that 'error' is actually 'data'. If you would like me to debug anything further, I'll be glad to. If it was Android I would have already done a pull request with a patch, but I'm not an iOS dev by any stretch of the imagination. |
Ah ok, now I got it! 💡 This issue is caused by the content type header "text/calendar". There is a white-list of allowed content types (see here) and "text/calendar" is not listed there. I think the white-list was meant to prevent problems with not supported data types, but it's only causing problems. Should be removed. I'll create a new issue referencing this one. |
I'm closing this issue now. Please feel free to re-open it, if #58 is not fixing your problem. |
I use cordova.plugin.http.post and it keep go to error response. ios 10 and below work fine.
The text was updated successfully, but these errors were encountered: