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

Bug in Safari/Chrome iOS #1389

Open
AlexNodex opened this issue Jul 11, 2016 · 7 comments
Open

Bug in Safari/Chrome iOS #1389

AlexNodex opened this issue Jul 11, 2016 · 7 comments

Comments

@AlexNodex
Copy link

AlexNodex commented Jul 11, 2016

I have found a very very strange bug in Safari and Chrome on iPad, iPhone and Simulator (Xcode)

On certain photo's (in my case it was photo's from an older iOS pre-update) the uploader will not let me choose a file. By this I mean that it will open the "choose files from "Take photo" and "Photo Library" but as soon as I choose (certain) photo's it fails.

This is where it gets strange. On the same devices it will allow me quite easily to upload other photos that are in the same "Photo library" - the difference being that they're newer images and not pre iOS update.

I have reproduced this on the Xcode simulator and it raises some interesting logs.

https://gist.github.com/AlexNodex/81048084b990bcbdc131f8f94bf71cf7

What's the most interesting is the following line
Jul 11 12:33:03 Alexs-MacBook-Pro com.apple.UIKit.fileprovider.default[1319]: Failed to inherit CoreMedia permissions from 1313: (null)

Which talks about permissions. I am perpelxed at what would cause this and how to go about fixing it. I have tried showing (removing the opacity of the button) and other fixes but nothing works.

@AlexNodex
Copy link
Author

Futher adding to this.

I have debugged some more and I get a "File extension error". It only happens on "image.jpeg" not "image.jpg" or "image.png". I've even set the filter to accept only "jpeg" and "*" but still no avail.

@James6xie
Copy link

On OSX safari9.0 upload file can not select!!

@jayarjo
Copy link
Contributor

jayarjo commented Aug 24, 2016

Still not sure about iOS bug (having trouble to replicate it), however @zhongtianxie your issue seems unrelated. Please open separate ticket and provide more information, since in general Plupload works ok in Safari 9.x on Mac OS X.

@jayarjo
Copy link
Contributor

jayarjo commented Aug 24, 2016

We now have a playground here: http://play.plupload.com/. So you can provide an exact code there to replicate your problem.

@wwwdata
Copy link

wwwdata commented May 29, 2017

I think the problem is here. It tries to detect the Browser and on iOS the Browser would be Mobile Safari but this does not match in this check function here: https://github.com/moxiecode/plupload/blob/master/js/moxie.js#L6134

summon_file_dialog: Test(function() { // yeah... some dirty sniffing here...
					return (Env.browser === 'Firefox' && Env.version >= 4) ||
						(Env.browser === 'Opera' && Env.version >= 12) ||
						(Env.browser === 'IE' && Env.version >= 10) ||
						!!~Basic.inArray(Env.browser, ['Chrome', 'Safari']);
				}()),

Basic.inArray seems to be an exact string check. This should be improved and modern mobile browser need to be added to it.

Edit: I'm not even sure this is the right Issue for it :P if not I can open another one

@renatocarvalho
Copy link

renatocarvalho commented Aug 27, 2017

+1

I'm having the same issue, where I cannot upload a photo from camera roll. I'll try to get back with more info.

@jayarjo
Copy link
Contributor

jayarjo commented Aug 27, 2017

If the cause for this is that issue that @wwwdata described, then it will be addressed in the bugfix release that is coming out on monday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants