cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From julio cesar sanchez <jcesarmob...@gmail.com>
Subject Re: [Android] Camera Plugin Request
Date Thu, 09 Jul 2015 08:54:06 GMT
I'm sorry, I think I got it wrong.
What is this pull request supposed to do?
I thought it will provide real urls instead of content:// urls, but I've
tried on a nexus 7 with 5.1.1 and moto g with 5.0.2 and I'm getting
content:// urls on most cases

So, what should we help you testing?
Do you have a sample and expected results?




2015-07-09 8:25 GMT+02:00 julio cesar sanchez <jcesarmobile@gmail.com>:

> But I think the FileHelper class should be moved somewhere else (not sure
> where), so other plugins can use it and not just the camera plugin.
> I have a file chooser plugin and I use very similar code to get the real
> urls
>
> 2015-07-08 23:52 GMT+02:00 Joe Bowser <bowserj@gmail.com>:
>
>> I just tried it and it doesn't work.  I think we should probably adopt
>> this, since I don't have a better solution to this problem.  I'll pull it
>> in tomorrow if I don't see any objections.
>>
>> On Wed, Jul 8, 2015 at 1:32 PM Joe Bowser <bowserj@gmail.com> wrote:
>>
>> > That makes sense.  I'll whitelist the Content URI and see if that
>> resolves
>> > the issue.  I do think we need something to get real URIs on Android but
>> > this seems like the wrong place for it.
>> >
>> > On Wed, Jul 8, 2015, 1:21 PM Nikhil Khandelwal <nikhilkh@microsoft.com>
>> > wrote:
>> >
>> >> I remember seeing a PR, that mentioned content:// was not working
>> because
>> >> of the whitelist plugin not supporting it when '*' was specified and/or
>> >> because CSP did not include it. That PR and discussion did not go
>> anywhere:
>> >> https://github.com/apache/cordova-plugin-whitelist/pull/6
>> >>
>> >> I have myself not tried the scenarios here and I wonder if they are
>> >> related.
>> >>
>> >> Thanks,
>> >> Nikhil
>> >>
>> >>
>> >> -----Original Message-----
>> >> From: Joe Bowser [mailto:bowserj@gmail.com]
>> >> Sent: Wednesday, July 8, 2015 11:42 AM
>> >> To: dev
>> >> Subject: [Android] Camera Plugin Request
>> >>
>> >> Hey
>> >>
>> >> Since for some reason the browser doesn't resolve content URIs anymore,
>> >> I'm tempted to pull this into Camera.
>> >>
>> >> https://github.com/apache/cordova-plugin-camera/pull/93
>> >>
>> >> The problem I have with it is the large amount of code that is being
>> >> added, and I want some more people to look at it before I OK it.  If
>> that
>> >> could happen, that'd be awesome.
>> >>
>> >> Thanks
>> >>
>> >
>>
>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message