[jira] [Commented] (CB-12551) File-Transfer download/upload fails in Android 4.4 or older using TLS 1.2

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12551?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634638#comment-16634638
 ] 

ASF GitHub Bot commented on CB-12551:
-

janpio commented on issue #197: CB-12551: (android) Patch Security Provider to 
support TLS 1.2
URL: 
https://github.com/apache/cordova-plugin-file-transfer/pull/197#issuecomment-426063702
 
 
   I just submitted a fix to master, if you rebase on that your tests might 
pass (or at least fail less).


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> File-Transfer download/upload fails in Android 4.4 or older using TLS 1.2
> -
>
> Key: CB-12551
> URL: https://issues.apache.org/jira/browse/CB-12551
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file-transfer (DEPRECATED)
>Reporter: Dani Palou
>Priority: Major
>
> FileTransfer's download function fails when trying to download a file from a 
> site using TLS 1.2. This works fine in Android 5 or higher, but it fails in 
> Android 4.x.
> I searched a bit and I found that Android 4.4 should support TLS1.2 but it is 
> disabled by default. I applied the solution proposed in this post and it 
> worked for me:
> http://stackoverflow.com/a/33567745
> That is, I created the _MySSLSocketFactory_ class in the FileTransfer 
> project, and I used it in the _trustAllHosts_ function (line 636 of 
> _FileTransfer.java_):
> {noformat}
> SSLSocketFactory newFactory = new MySSLSocketFactory(sc.getSocketFactory());
> {noformat}
> I'm not sure if this can cause problems with other certificates, I don't know 
> much about this. It would be nice if the Cordova team could take a look at 
> this and check if it can be solved.
> This is the stacktrace:
> {noformat}
> Error getting HTTP status code from connection.
> javax.net.ssl.SSLException: Connection closed by peer
>  at com.android.org.conscrypt.NativeCrypto.SSL_do_handshake(Native Method)
>  at 
> com.android.org.conscrypt.OpenSSLSocketImpl.startHandshake(OpenSSLSocketImpl.java:405)
>  at com.android.okhttp.Connection.upgradeToTls(Connection.java:146)
>  at com.android.okhttp.Connection.connect(Connection.java:107)
>  at com.android.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:294)
>  at 
> com.android.okhttp.internal.http.HttpEngine.sendSocketRequest(HttpEngine.java:255)
>  at 
> com.android.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:206)
>  at 
> com.android.okhttp.internal.http.HttpURLConnectionImpl.execute(HttpURLConnectionImpl.java:345)
>  at 
> com.android.okhttp.internal.http.HttpURLConnectionImpl.connect(HttpURLConnectionImpl.java:89)
>  at 
> com.android.okhttp.internal.http.HttpsURLConnectionImpl.connect(HttpsURLConnectionImpl.java:161)
>  at org.apache.cordova.filetransfer.FileTransfer$4.run(FileTransfer.java:869)
>  at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1112)
>  at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:587)
>  at java.lang.Thread.run(Thread.java:841)
> {noformat}
> It can be tested with this code:
> {noformat}
> var ft = new FileTransfer(),
> path = 'squirrel.jpg',
> uri = 
> 'https://prototype.moodle.net/mobile/moodle32/webservice/pluginfile.php/314/mod_resource/content/3/squirrel.jpg?forcedownload=1=616455be9f363cc9631cab89cfcfa1cd';
> ft.download(uri, path, success, error, true);
> function success() {
> console.log('Success');
> }
> function error(data) {
> console.log('Error', data);
> }
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13690) Change audio stream src in platform-browser

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13690?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634619#comment-16634619
 ] 

ASF GitHub Bot commented on CB-13690:
-

janpio commented on issue #159: CB-13690 : (browser) Add src change to method 
to Media object
URL: 
https://github.com/apache/cordova-plugin-media/pull/159#issuecomment-426056733
 
 
   Tests are now passing.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Change audio stream src in platform-browser
> ---
>
> Key: CB-13690
> URL: https://issues.apache.org/jira/browse/CB-13690
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-media
>Affects Versions: 4.0.0
> Environment: Browser
>Reporter: Frank Owusu
>Priority: Critical
>  Labels: Audio, browser, easyfix, patch, stream
> Fix For: 4.0.0
>
>
> h2. Media object change src method for Browser.
> when node is released stream src is still being downloaded.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13850) Typescript: Misspelled property fileSystem

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634576#comment-16634576
 ] 

ASF GitHub Bot commented on CB-13850:
-

janpio closed pull request #229: CB-13850: Fix spelling in typed-file: property 
name filesystem (wrong…
URL: https://github.com/apache/cordova-plugin-file/pull/229
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/types/index.d.ts b/types/index.d.ts
index c748e3da..ba17a43d 100644
--- a/types/index.d.ts
+++ b/types/index.d.ts
@@ -63,7 +63,7 @@ interface Entry {
 /** The full absolute path from the root to the entry. */
 fullPath: string;
 /** The file system on which the entry resides. */
-fileSystem: FileSystem;
+filesystem: FileSystem;
 nativeURL: string;
 /**
  * Look up metadata about this entry.


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Typescript: Misspelled property fileSystem
> --
>
> Key: CB-13850
> URL: https://issues.apache.org/jira/browse/CB-13850
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Reporter: Tobias Haupenthal
>Priority: Major
>
> The Typescript file index.d.ts shows that an Entry shall have an attribute 
> fileSystem.
> But in Android this property is undefined. There is only the property 
> filesystem.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13850) Typescript: Misspelled property fileSystem

2018-10-01 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13850?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634577#comment-16634577
 ] 

ASF subversion and git services commented on CB-13850:
--

Commit 4642fde47ff24f6a2a38ef5f812cca04e1c23a1a in cordova-plugin-file's branch 
refs/heads/master from Beutlin
[ https://gitbox.apache.org/repos/asf?p=cordova-plugin-file.git;h=4642fde ]

CB-13850: Fix spelling in typed-file: property name filesystem (wrong… (#229)



### Platforms affected
-

### What does this PR do?
The DefinitelyTyped file has a wrong attribute for an Entry object. It is 
"fileSystem" but should be "filesystem". The consequence of this is that the 
syntax checker of an IDE might give an false warning.

### What testing has been done on this change?
Looked at file www/Entry.js. Obviously the naming is here "filesystem".

### Checklist
- [ x ] [Reported an issue](http://cordova.apache.org/contribute/issues.html) 
in the JIRA database
- [ x ] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB- is the JIRA ID & "android" is the platform 
affected.
- Automated tests are not possible, because this bug is related to a 
DefinitelyTyped file, which provides only syntax checking for IDEs.


> Typescript: Misspelled property fileSystem
> --
>
> Key: CB-13850
> URL: https://issues.apache.org/jira/browse/CB-13850
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Reporter: Tobias Haupenthal
>Priority: Major
>
> The Typescript file index.d.ts shows that an Entry shall have an attribute 
> fileSystem.
> But in Android this property is undefined. There is only the property 
> filesystem.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13960) Typescript: FileWriter write argument definition shall include type string

2018-10-01 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634575#comment-16634575
 ] 

ASF subversion and git services commented on CB-13960:
--

Commit 5353b847270d55f8caedfc992393137514c59992 in cordova-plugin-file's branch 
refs/heads/master from vldmr-bus
[ https://gitbox.apache.org/repos/asf?p=cordova-plugin-file.git;h=5353b84 ]

CB-13960: fix FileWriter.write argument type definition for typescript (#231)

Fix FileWriter.write argument type definition for typescript


> Typescript: FileWriter write argument definition shall include type string
> --
>
> Key: CB-13960
> URL: https://issues.apache.org/jira/browse/CB-13960
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Reporter: Vladimir
>Priority: Major
>
> www\FileWriter.js has the following annotation on write method:
> {code:java}
> @param data text or blob to be writte{code}
> But types\index.d.ts has the following type definition:
> {code:java}
> /**
>  * Write the supplied data to the file at position.
>  * @param {Blob} data The blob to write.
>  */
> write(data: Blob): void;{code}
> Shall be as follows:
> {code:java}
> /**
>  * Write the supplied data to the file at position.
>  * @param {Blob|string} data The blob to write.
>  */
> write(data: Blob|string): void;{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13960) Typescript: FileWriter write argument definition shall include type string

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13960?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634574#comment-16634574
 ] 

ASF GitHub Bot commented on CB-13960:
-

janpio closed pull request #231: CB-13960: fix FileWriter.write argument type 
definition for typescript
URL: https://github.com/apache/cordova-plugin-file/pull/231
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/types/index.d.ts b/types/index.d.ts
index c748e3da..40f1a822 100644
--- a/types/index.d.ts
+++ b/types/index.d.ts
@@ -294,9 +294,9 @@ interface FileWriter extends FileSaver {
 length: number;
 /**
  * Write the supplied data to the file at position.
- * @param {Blob} data The blob to write.
+ * @param {Blob|string} data The blob to write.
  */
-write(data: Blob): void;
+write(data: Blob|string): void;
 /**
  * The file position at which the next write will occur.
  * @param offset If nonnegative, an absolute byte offset into the file.


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Typescript: FileWriter write argument definition shall include type string
> --
>
> Key: CB-13960
> URL: https://issues.apache.org/jira/browse/CB-13960
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Reporter: Vladimir
>Priority: Major
>
> www\FileWriter.js has the following annotation on write method:
> {code:java}
> @param data text or blob to be writte{code}
> But types\index.d.ts has the following type definition:
> {code:java}
> /**
>  * Write the supplied data to the file at position.
>  * @param {Blob} data The blob to write.
>  */
> write(data: Blob): void;{code}
> Shall be as follows:
> {code:java}
> /**
>  * Write the supplied data to the file at position.
>  * @param {Blob|string} data The blob to write.
>  */
> write(data: Blob|string): void;{code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-11714) Camera plugin for windows does not properly set content-type when selecting JPG->PNG

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-11714?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634551#comment-16634551
 ] 

ASF GitHub Bot commented on CB-11714:
-

janpio commented on issue #242: CB-11714: (windows) added check for encoding in 
savePhoto() without height/width
URL: 
https://github.com/apache/cordova-plugin-camera/pull/242#issuecomment-426043765
 
 
   Thanks @DisruptiveMind.
   
   Tests are now failing because of
   ```
   /Users/travis/build/apache/cordova-plugin-camera/src/windows/CameraProxy.js
 790:67  error  Strings must use singlequote  quotes
   ✖ 1 problem (1 error, 0 warnings)
 1 error, 0 warnings potentially fixable with the `--fix` option.
   ```
   This doesn't make too much sense to me, especially as line 790 contains no 
quotes at all.
   
   Can you maybe rebase on `master`? I hope this will go away with more recent 
test configuration. Thanks!


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Camera plugin for windows does not properly set content-type when selecting 
> JPG->PNG
> 
>
> Key: CB-11714
> URL: https://issues.apache.org/jira/browse/CB-11714
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-camera
>Affects Versions: Master
> Environment: Windows 10
>Reporter: Tyler Pham
>Priority: Major
>  Labels: easyfix, triaged, windows
>
> When selecting a JPG with encodingType = Camera.EncodingType.PNG, the 
> resizeImage() will not modify the target content type when converting 
> canvas.toDataURL(...) -- the content-type will still be image/jpeg but with a 
> .png extension which will cause the browser to not render to image and issue 
> an error.
> Possibly related to: CB-8618
> I have created a small fix at:
> https://github.com/DisruptiveMind/cordova-plugin-camera/tree/fix-windows-content-type



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-7179) [InAppBrowser][iOS 8] Update to support WKWebView

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-7179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634540#comment-16634540
 ] 

ASF GitHub Bot commented on CB-7179:


janpio edited a comment on issue #271: CB-7179 (iOS): Add WKWebView support for 
iOS
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/271#issuecomment-425728887
 
 
   Simplest answer: 
   Because the testing setup for plugins is all messed up, see a WIP issue 
here: https://github.com/apache/cordova/issues/28
   I am working my way through these, but it is _very_ slow going.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [InAppBrowser][iOS 8] Update to support WKWebView
> -
>
> Key: CB-7179
> URL: https://issues.apache.org/jira/browse/CB-7179
> Project: Apache Cordova
>  Issue Type: Sub-task
>  Components: cordova-plugin-inappbrowser
> Environment: iOS 8
>Reporter: Shazron Abdullah
>Priority: Major
>
> support dual use with UIWebView



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12035) [cordova-plugin-network-information] connection info is not reliable on Android 6

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634526#comment-16634526
 ] 

ASF GitHub Bot commented on CB-12035:
-

PieterVanPoyer edited a comment on issue #74: CB-12035 (android) Fix bug 
[cordova-plugin-network-information] connection info is not reliable on Android 
6
URL: 
https://github.com/apache/cordova-plugin-network-information/pull/74#issuecomment-426035590
 
 
   I triggered a new build. (I did change some logging for a new commit).
   The travis build is now successfull!
   Are there any comments, questions, remarks for this pull request?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [cordova-plugin-network-information] connection info is not reliable on 
> Android 6
> -
>
> Key: CB-12035
> URL: https://issues.apache.org/jira/browse/CB-12035
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-network-information
>Affects Versions: 1.3.0
> Environment: Samsung Galaxy Edge S6 / Android 6
> useragent: "Mozilla/5.0 (Linux; Android 6.0.1; SM-G925F Build/MMB29K; wv) 
> AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/53.0.2785.124 
> Mobile Safari/537.36"
> reproduced with:
> cordova-plugin-network-information@1.3.0 (latest)
> cordova-plugin-network-information@1.2.0
>Reporter: jakub-g
>Priority: Major
>  Labels: android
>
> I've noticed that information exposed by the plugin is not reliable. It 
> happens often that I'm connected to the internet via WiFi, and the plugin 
> says that there's no connection.
> It's not easy to reproduce it on-demand, but I reproduced it many times 
> recently. It happens most often when in the morning, I open the Cordova app 
> that was running in background throughout the night.
> Then, when I connect to Chrome Dev Tools, I can do a repl session like this:
> {code}
> > navigator.connection.type
> "none"
> > window.fetch('http://www.example.org').then(function(res) 
> > {console.log(res.status)})
> Promise {[[PromiseStatus]]: "pending", [[PromiseValue]]: undefined}
> 200
> Fetch complete: GET "http://www.example.org/;.(anonymous function)
> > navigator.connection.type
> "none"
> {code}
> So, the plugin thinks I'm offline, yet any HTTP calls done within the app 
> without checking if I'm offline or online, are successful (either from the 
> app or devtools).
> When I disable WiFi and reenable it, after a few seconds the plugin updates 
> its state and then it tells that connection type is "wifi".
> There are several possibilities here:
> - either system (or device driver) gives us incorrect information, or
> - cordova plugin is caching data and/or not listening properly to all the 
> events and/or incorrectly processing it
> and most probably the issue manifests itself when the device is asleep for 
> many hours (but I'm not sure about this).
> Note that I also noticed another issue, that sometimes the Chrome network 
> stack thinks I'm offline (when I do an HTTP call, it fails immediately and I 
> see network errors in devtools console) - and then I restart the Cordova app, 
> and everything's fine, all HTTP calls are ok - but this is unrelated to this 
> ticket, and perhaps a bug in Chrome.
> This ticket seems to be most likely a bug in the plugin, because as I said 
> before, doing HTTP call succeeds despite the plugin telling me that I'm 
> offline.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12035) [cordova-plugin-network-information] connection info is not reliable on Android 6

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634525#comment-16634525
 ] 

ASF GitHub Bot commented on CB-12035:
-

PieterVanPoyer commented on issue #74: CB-12035 (android) Fix bug 
[cordova-plugin-network-information] connection info is not reliable on Android 
6
URL: 
https://github.com/apache/cordova-plugin-network-information/pull/74#issuecomment-426035590
 
 
   I triggered a new build. (I did change some logging for a new commit).
   Are there any comments, questions, remarks for this pull request?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [cordova-plugin-network-information] connection info is not reliable on 
> Android 6
> -
>
> Key: CB-12035
> URL: https://issues.apache.org/jira/browse/CB-12035
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-network-information
>Affects Versions: 1.3.0
> Environment: Samsung Galaxy Edge S6 / Android 6
> useragent: "Mozilla/5.0 (Linux; Android 6.0.1; SM-G925F Build/MMB29K; wv) 
> AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/53.0.2785.124 
> Mobile Safari/537.36"
> reproduced with:
> cordova-plugin-network-information@1.3.0 (latest)
> cordova-plugin-network-information@1.2.0
>Reporter: jakub-g
>Priority: Major
>  Labels: android
>
> I've noticed that information exposed by the plugin is not reliable. It 
> happens often that I'm connected to the internet via WiFi, and the plugin 
> says that there's no connection.
> It's not easy to reproduce it on-demand, but I reproduced it many times 
> recently. It happens most often when in the morning, I open the Cordova app 
> that was running in background throughout the night.
> Then, when I connect to Chrome Dev Tools, I can do a repl session like this:
> {code}
> > navigator.connection.type
> "none"
> > window.fetch('http://www.example.org').then(function(res) 
> > {console.log(res.status)})
> Promise {[[PromiseStatus]]: "pending", [[PromiseValue]]: undefined}
> 200
> Fetch complete: GET "http://www.example.org/;.(anonymous function)
> > navigator.connection.type
> "none"
> {code}
> So, the plugin thinks I'm offline, yet any HTTP calls done within the app 
> without checking if I'm offline or online, are successful (either from the 
> app or devtools).
> When I disable WiFi and reenable it, after a few seconds the plugin updates 
> its state and then it tells that connection type is "wifi".
> There are several possibilities here:
> - either system (or device driver) gives us incorrect information, or
> - cordova plugin is caching data and/or not listening properly to all the 
> events and/or incorrectly processing it
> and most probably the issue manifests itself when the device is asleep for 
> many hours (but I'm not sure about this).
> Note that I also noticed another issue, that sometimes the Chrome network 
> stack thinks I'm offline (when I do an HTTP call, it fails immediately and I 
> see network errors in devtools console) - and then I restart the Cordova app, 
> and everything's fine, all HTTP calls are ok - but this is unrelated to this 
> ticket, and perhaps a bug in Chrome.
> This ticket seems to be most likely a bug in the plugin, because as I said 
> before, doing HTTP call succeeds despite the plugin telling me that I'm 
> offline.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12035) [cordova-plugin-network-information] connection info is not reliable on Android 6

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634506#comment-16634506
 ] 

ASF GitHub Bot commented on CB-12035:
-

PieterVanPoyer removed a comment on issue #74: CB-12035 (android) Fix bug 
[cordova-plugin-network-information] connection info is not reliable on Android 
6
URL: 
https://github.com/apache/cordova-plugin-network-information/pull/74#issuecomment-426029809
 
 
   @janpio Hey janpio, the travis-ci build keeps failing. Can I restart the 
travis build?  
   I 've seen that 
https://github.com/apache/cordova-plugin-media-capture/issues/105 is closed 
now. 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [cordova-plugin-network-information] connection info is not reliable on 
> Android 6
> -
>
> Key: CB-12035
> URL: https://issues.apache.org/jira/browse/CB-12035
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-network-information
>Affects Versions: 1.3.0
> Environment: Samsung Galaxy Edge S6 / Android 6
> useragent: "Mozilla/5.0 (Linux; Android 6.0.1; SM-G925F Build/MMB29K; wv) 
> AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/53.0.2785.124 
> Mobile Safari/537.36"
> reproduced with:
> cordova-plugin-network-information@1.3.0 (latest)
> cordova-plugin-network-information@1.2.0
>Reporter: jakub-g
>Priority: Major
>  Labels: android
>
> I've noticed that information exposed by the plugin is not reliable. It 
> happens often that I'm connected to the internet via WiFi, and the plugin 
> says that there's no connection.
> It's not easy to reproduce it on-demand, but I reproduced it many times 
> recently. It happens most often when in the morning, I open the Cordova app 
> that was running in background throughout the night.
> Then, when I connect to Chrome Dev Tools, I can do a repl session like this:
> {code}
> > navigator.connection.type
> "none"
> > window.fetch('http://www.example.org').then(function(res) 
> > {console.log(res.status)})
> Promise {[[PromiseStatus]]: "pending", [[PromiseValue]]: undefined}
> 200
> Fetch complete: GET "http://www.example.org/;.(anonymous function)
> > navigator.connection.type
> "none"
> {code}
> So, the plugin thinks I'm offline, yet any HTTP calls done within the app 
> without checking if I'm offline or online, are successful (either from the 
> app or devtools).
> When I disable WiFi and reenable it, after a few seconds the plugin updates 
> its state and then it tells that connection type is "wifi".
> There are several possibilities here:
> - either system (or device driver) gives us incorrect information, or
> - cordova plugin is caching data and/or not listening properly to all the 
> events and/or incorrectly processing it
> and most probably the issue manifests itself when the device is asleep for 
> many hours (but I'm not sure about this).
> Note that I also noticed another issue, that sometimes the Chrome network 
> stack thinks I'm offline (when I do an HTTP call, it fails immediately and I 
> see network errors in devtools console) - and then I restart the Cordova app, 
> and everything's fine, all HTTP calls are ok - but this is unrelated to this 
> ticket, and perhaps a bug in Chrome.
> This ticket seems to be most likely a bug in the plugin, because as I said 
> before, doing HTTP call succeeds despite the plugin telling me that I'm 
> offline.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12035) [cordova-plugin-network-information] connection info is not reliable on Android 6

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12035?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634505#comment-16634505
 ] 

ASF GitHub Bot commented on CB-12035:
-

PieterVanPoyer commented on issue #74: CB-12035 (android) Fix bug 
[cordova-plugin-network-information] connection info is not reliable on Android 
6
URL: 
https://github.com/apache/cordova-plugin-network-information/pull/74#issuecomment-426029809
 
 
   @janpio Hey janpio, the travis-ci build keeps failing. Can I restart the 
travis build?  
   I 've seen that 
https://github.com/apache/cordova-plugin-media-capture/issues/105 is closed 
now. 
   


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [cordova-plugin-network-information] connection info is not reliable on 
> Android 6
> -
>
> Key: CB-12035
> URL: https://issues.apache.org/jira/browse/CB-12035
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-network-information
>Affects Versions: 1.3.0
> Environment: Samsung Galaxy Edge S6 / Android 6
> useragent: "Mozilla/5.0 (Linux; Android 6.0.1; SM-G925F Build/MMB29K; wv) 
> AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/53.0.2785.124 
> Mobile Safari/537.36"
> reproduced with:
> cordova-plugin-network-information@1.3.0 (latest)
> cordova-plugin-network-information@1.2.0
>Reporter: jakub-g
>Priority: Major
>  Labels: android
>
> I've noticed that information exposed by the plugin is not reliable. It 
> happens often that I'm connected to the internet via WiFi, and the plugin 
> says that there's no connection.
> It's not easy to reproduce it on-demand, but I reproduced it many times 
> recently. It happens most often when in the morning, I open the Cordova app 
> that was running in background throughout the night.
> Then, when I connect to Chrome Dev Tools, I can do a repl session like this:
> {code}
> > navigator.connection.type
> "none"
> > window.fetch('http://www.example.org').then(function(res) 
> > {console.log(res.status)})
> Promise {[[PromiseStatus]]: "pending", [[PromiseValue]]: undefined}
> 200
> Fetch complete: GET "http://www.example.org/;.(anonymous function)
> > navigator.connection.type
> "none"
> {code}
> So, the plugin thinks I'm offline, yet any HTTP calls done within the app 
> without checking if I'm offline or online, are successful (either from the 
> app or devtools).
> When I disable WiFi and reenable it, after a few seconds the plugin updates 
> its state and then it tells that connection type is "wifi".
> There are several possibilities here:
> - either system (or device driver) gives us incorrect information, or
> - cordova plugin is caching data and/or not listening properly to all the 
> events and/or incorrectly processing it
> and most probably the issue manifests itself when the device is asleep for 
> many hours (but I'm not sure about this).
> Note that I also noticed another issue, that sometimes the Chrome network 
> stack thinks I'm offline (when I do an HTTP call, it fails immediately and I 
> see network errors in devtools console) - and then I restart the Cordova app, 
> and everything's fine, all HTTP calls are ok - but this is unrelated to this 
> ticket, and perhaps a bug in Chrome.
> This ticket seems to be most likely a bug in the plugin, because as I said 
> before, doing HTTP call succeeds despite the plugin telling me that I'm 
> offline.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13307) Accessing Google Photos on Pixel from cordova-plugin-camera

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634485#comment-16634485
 ] 

ASF GitHub Bot commented on CB-13307:
-

janpio commented on issue #286: CB-13307: (Android) Fixed issue when unable to 
load image with mime t…
URL: 
https://github.com/apache/cordova-plugin-camera/pull/286#issuecomment-426024101
 
 
   Jippieh! I removed some unwanted changes to README that somehow made their 
way into this via merge/rebase. Now "only" waiting for review and merge.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Accessing Google Photos on Pixel from cordova-plugin-camera
> ---
>
> Key: CB-13307
> URL: https://issues.apache.org/jira/browse/CB-13307
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-camera
>Reporter: David Briglio
>Priority: Major
>
> Currently when trying to access some photos from Google Photos, it fails to 
> load. Android 8.0 on Google Pixel, cordova version 6.5.0, 
> cordova-plugin-camera version 2.4.1.
> Causes:
> 1) Only mimetypes "image/jpeg" and "image/png" are accepted, all other types 
> are considered an invalid image. The mimetype of the image I am loading gets 
> set to "image/*".
> 2) In FileHelper.java: when hitting the isGooglePhotosUri method, the uri 
> authority is compared against "com.google.android.apps.photos.content", where 
> the actual is "com.google.android.apps.photos.contentprovider". This causes 
> the logic to dismiss the image as not from Google Photos.
> After applying fixes for these two areas, I am able to successfully load all 
> of my images.
> Merge request [here|https://github.com/apache/cordova-plugin-camera/pull/286].



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13307) Accessing Google Photos on Pixel from cordova-plugin-camera

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634475#comment-16634475
 ] 

ASF GitHub Bot commented on CB-13307:
-

DavidBriglio commented on issue #286: CB-13307: (Android) Fixed issue when 
unable to load image with mime t…
URL: 
https://github.com/apache/cordova-plugin-camera/pull/286#issuecomment-426020724
 
 
   @janpio Tests are now passing!


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Accessing Google Photos on Pixel from cordova-plugin-camera
> ---
>
> Key: CB-13307
> URL: https://issues.apache.org/jira/browse/CB-13307
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-camera
>Reporter: David Briglio
>Priority: Major
>
> Currently when trying to access some photos from Google Photos, it fails to 
> load. Android 8.0 on Google Pixel, cordova version 6.5.0, 
> cordova-plugin-camera version 2.4.1.
> Causes:
> 1) Only mimetypes "image/jpeg" and "image/png" are accepted, all other types 
> are considered an invalid image. The mimetype of the image I am loading gets 
> set to "image/*".
> 2) In FileHelper.java: when hitting the isGooglePhotosUri method, the uri 
> authority is compared against "com.google.android.apps.photos.content", where 
> the actual is "com.google.android.apps.photos.contentprovider". This causes 
> the logic to dismiss the image as not from Google Photos.
> After applying fixes for these two areas, I am able to successfully load all 
> of my images.
> Merge request [here|https://github.com/apache/cordova-plugin-camera/pull/286].



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13307) Accessing Google Photos on Pixel from cordova-plugin-camera

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13307?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634379#comment-16634379
 ] 

ASF GitHub Bot commented on CB-13307:
-

janpio commented on issue #286: CB-13307: (Android) Fixed issue when unable to 
load image with mime t…
URL: 
https://github.com/apache/cordova-plugin-camera/pull/286#issuecomment-425996898
 
 
   And another rebase/merge from `master` to get those tests passing 
@DavidBriglio?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Accessing Google Photos on Pixel from cordova-plugin-camera
> ---
>
> Key: CB-13307
> URL: https://issues.apache.org/jira/browse/CB-13307
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-camera
>Reporter: David Briglio
>Priority: Major
>
> Currently when trying to access some photos from Google Photos, it fails to 
> load. Android 8.0 on Google Pixel, cordova version 6.5.0, 
> cordova-plugin-camera version 2.4.1.
> Causes:
> 1) Only mimetypes "image/jpeg" and "image/png" are accepted, all other types 
> are considered an invalid image. The mimetype of the image I am loading gets 
> set to "image/*".
> 2) In FileHelper.java: when hitting the isGooglePhotosUri method, the uri 
> authority is compared against "com.google.android.apps.photos.content", where 
> the actual is "com.google.android.apps.photos.contentprovider". This causes 
> the logic to dismiss the image as not from Google Photos.
> After applying fixes for these two areas, I am able to successfully load all 
> of my images.
> Merge request [here|https://github.com/apache/cordova-plugin-camera/pull/286].



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12873) Options needed to customize caputure button

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634353#comment-16634353
 ] 

ASF GitHub Bot commented on CB-12873:
-

charlesverge commented on issue #268: CB-12873: Add customization options for 
caputre button, resizing of v…
URL: 
https://github.com/apache/cordova-plugin-camera/pull/268#issuecomment-425989962
 
 
   @janpio I've rebased the commit, the changes since the last rebase do not 
affect this code and everything looks good.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Options needed to customize caputure button
> ---
>
> Key: CB-12873
> URL: https://issues.apache.org/jira/browse/CB-12873
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-camera
>Affects Versions: 6.2.2
> Environment: browser
>Reporter: Charles Verge
>Priority: Major
>  Labels: browser, camera
>
> The browser version of the camera plugin is missing
> # Options to configure
> ## text for capture button
> ## Class
> ## Cancel button
> ## zIndex
> ## position
> # Cancel button
> # Resizing of video upon browser resize or orientation change
> # Graceful activation of file selector dialog and visibility of file input



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12479) InAppBrowser: Add Loading Spinner for Android

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634146#comment-16634146
 ] 

ASF GitHub Bot commented on CB-12479:
-

cjpearson commented on issue #88: CB-12479: Add loading spinner for android
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/88#issuecomment-425937808
 
 
   @janpio No problem, I had to make another change anyway. The latest commit 
replaces the deprecated ProgressDialog with a ProgressBar.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> InAppBrowser: Add Loading Spinner for Android
> -
>
> Key: CB-12479
> URL: https://issues.apache.org/jira/browse/CB-12479
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-inappbrowser
>Affects Versions: 2.5.0, 2.6.0, 2.7.0, 2.8.0
> Environment: Android (at least)
>Reporter: jcesarmobile
>Priority: Trivial
>  Labels: Triaged
>
> In CB-3750 a loader spinner was added for iOS. There was another PR for 
> Android but it was never merged, so I'm creating this new issue for Android.
> PR https://github.com/apache/cordova-plugin-inappbrowser/pull/88



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12479) InAppBrowser: Add Loading Spinner for Android

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634115#comment-16634115
 ] 

ASF GitHub Bot commented on CB-12479:
-

janpio commented on issue #88: CB-12479: Add loading spinner for android
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/88#issuecomment-425935160
 
 
   Thanks for the rebase, and sorry I didn't check what repo this PR was in 
before writing this: `cordova-plugin-inappbrowser` has another test problem. 
You can follow my progress understanding it here: 
https://github.com/apache/cordova-plugin-inappbrowser/issues/307


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> InAppBrowser: Add Loading Spinner for Android
> -
>
> Key: CB-12479
> URL: https://issues.apache.org/jira/browse/CB-12479
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-inappbrowser
>Affects Versions: 2.5.0, 2.6.0, 2.7.0, 2.8.0
> Environment: Android (at least)
>Reporter: jcesarmobile
>Priority: Trivial
>  Labels: Triaged
>
> In CB-3750 a loader spinner was added for iOS. There was another PR for 
> Android but it was never merged, so I'm creating this new issue for Android.
> PR https://github.com/apache/cordova-plugin-inappbrowser/pull/88



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12479) InAppBrowser: Add Loading Spinner for Android

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12479?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634051#comment-16634051
 ] 

ASF GitHub Bot commented on CB-12479:
-

janpio commented on issue #88: CB-12479: Add loading spinner for android
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/88#issuecomment-425913364
 
 
   Awesome. Unfortunately I have some more "work" for you:
   I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> InAppBrowser: Add Loading Spinner for Android
> -
>
> Key: CB-12479
> URL: https://issues.apache.org/jira/browse/CB-12479
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-inappbrowser
>Affects Versions: 2.5.0, 2.6.0, 2.7.0, 2.8.0
> Environment: Android (at least)
>Reporter: jcesarmobile
>Priority: Trivial
>  Labels: Triaged
>
> In CB-3750 a loader spinner was added for iOS. There was another PR for 
> Android but it was never merged, so I'm creating this new issue for Android.
> PR https://github.com/apache/cordova-plugin-inappbrowser/pull/88



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-14024) Doesn't have into account statusbar height when toolbar is enabled

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-14024?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634050#comment-16634050
 ] 

ASF GitHub Bot commented on CB-14024:
-

janpio commented on issue #265: CB-14024 Take into account statusbar height 
when toolbar is enabled
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/265#issuecomment-425913163
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Doesn't have into account statusbar height when toolbar is enabled
> --
>
> Key: CB-14024
> URL: https://issues.apache.org/jira/browse/CB-14024
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-inappbrowser
>Affects Versions: cordova-ios 4.5.0
>Reporter: Humberto Alfonso Díaz
>Priority: Major
>  Labels: inappbrowser, ios
>
> When using the following configuration:
> {code:java}
> this.browser.create(element.url, '_blank',"toolbar=yes,toolbarposition=top");
> {code}
> the plugin doesn't takes into account he statusbar height causing the webpage 
> to not show properly on the header.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-14188) Callback to decide whether to open link or not.

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-14188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634049#comment-16634049
 ] 

ASF GitHub Bot commented on CB-14188:
-

janpio commented on issue #276: CB-14188: add beforeload event, catching 
navigation before it happens
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/276#issuecomment-425913068
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Callback to decide whether to open link or not.
> ---
>
> Key: CB-14188
> URL: https://issues.apache.org/jira/browse/CB-14188
> Project: Apache Cordova
>  Issue Type: New Feature
>  Components: cordova-plugin-inappbrowser
>Affects Versions: 3.0.0
>Reporter: wvengen
>Priority: Major
>
> Several times I've found myself wanting to tell inAppBrowser whether it can 
> open a link in the embedded webview or using the system web browser. It would 
> be great if there's a callback _before_ loading a page (loadstart is too late 
> there), where I can tell inAppBrowser to load the page or not.
> CB-14013 introduced AllowedSchemes, but excluded the use of http and https. 
> -It would be useful to allow intercepting http and https links, e.g. to 
> decide whether to open the link in the system web browser or not.- At first I 
> thought this mechanism could be used, but because of the asynchronous nature 
> of Cordova callbacks, this didn't work out, and I added a beforeload 
> callback. See comment for an example.
> I've submitted a PR -(my approach would be to generate a regexp based on 
> AllowedSchemes and match the URL on that, instead of the current if logic)-.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-14190) Double click `` element causes app to exit.

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-14190?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634048#comment-16634048
 ] 

ASF GitHub Bot commented on CB-14190:
-

janpio commented on issue #275: CB-14190: (windows) Fix bug where double 
clicking `` exits app
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/275#issuecomment-425913000
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Double click `` element causes app to exit.
> ---
>
> Key: CB-14190
> URL: https://issues.apache.org/jira/browse/CB-14190
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-inappbrowser, cordova-windows
>Affects Versions: 3.0.0
>Reporter: Iain Connor
>Assignee: Jesse MacFadyen
>Priority: Minor
>
> In Windows, create an inappbrowser and browse somewhere with target not 
> `_system` or `_self` or empty.
> Find a `` element and double click it.
> Expected behaviour:
> First click expands `` dropdown by showing ``s.
> Second click is ignored or chooses `` under cursor, if there is one.
> Actual behaviour:
> Second click exits the inappbrowser.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-12873) Options needed to customize caputure button

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-12873?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634046#comment-16634046
 ] 

ASF GitHub Bot commented on CB-12873:
-

janpio commented on issue #268: CB-12873: Add customization options for caputre 
button, resizing of v…
URL: 
https://github.com/apache/cordova-plugin-camera/pull/268#issuecomment-425912660
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Options needed to customize caputure button
> ---
>
> Key: CB-12873
> URL: https://issues.apache.org/jira/browse/CB-12873
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-camera
>Affects Versions: 6.2.2
> Environment: browser
>Reporter: Charles Verge
>Priority: Major
>  Labels: browser, camera
>
> The browser version of the camera plugin is missing
> # Options to configure
> ## text for capture button
> ## Class
> ## Cancel button
> ## zIndex
> ## position
> # Cancel button
> # Resizing of video upon browser resize or orientation change
> # Graceful activation of file selector dialog and visibility of file input



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13245) Filereader.readAsText fails on Android with long text files when

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13245?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634041#comment-16634041
 ] 

ASF GitHub Bot commented on CB-13245:
-

janpio commented on issue #217: CB-13245: (android) Fix bug caused by negative 
length reported for an asset
URL: 
https://github.com/apache/cordova-plugin-file/pull/217#issuecomment-425912429
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Filereader.readAsText fails on Android with long text files when
> 
>
> Key: CB-13245
> URL: https://issues.apache.org/jira/browse/CB-13245
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Affects Versions: Master
> Environment: Android 7.1.1
>Reporter: Daniel Behnen
>Priority: Major
>
> When reader.readAsText is called with an URI pointing to a local asset with a 
> size greater than READ_CHUNK_SIZE = 256 * 1024, loading produces corrupt 
> data. 
> The behaviour is caused by a negative length returned by 
> CordovaResourceApi.openForRead() when the URI is a URI_TYPE_ASSET. Given the 
> negative length, Filesystem.readFileAtURL() returns the whole file instead of 
> the desired part and FileReader.readSuccessCallback() does not check the 
> returned buffer size. Hence, the following chunks are attated to the buffer 
> regardless of whether the file was already completely read.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13208) File plugin on Windows reads entire file into memory when only a small fragment is requested on `readAsBinaryString`

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13208?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634039#comment-16634039
 ] 

ASF GitHub Bot commented on CB-13208:
-

janpio commented on issue #216: CB-13208: (windows) Fix `readAsBinaryString` to 
only read the requested range
URL: 
https://github.com/apache/cordova-plugin-file/pull/216#issuecomment-425912291
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> File plugin on Windows reads entire file into memory when only a small 
> fragment is requested on `readAsBinaryString`
> 
>
> Key: CB-13208
> URL: https://issues.apache.org/jira/browse/CB-13208
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Affects Versions: 6.5.0
> Environment: All Windows UAP applications
>Reporter: Samer Albahra
>Priority: Major
>
> `readAsBinaryString` for `FileReader` on Windows is reading the entire file 
> to a buffer then slicing the requested fragment causing the app to crash due 
> to out of memory errors. It should instead be using a stream which seeks to 
> the requested start position and only reads the requested length.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13836) Windows phone correct filename

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13836?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634044#comment-16634044
 ] 

ASF GitHub Bot commented on CB-13836:
-

janpio commented on issue #311: CB-13836 Windows phone correct filename
URL: 
https://github.com/apache/cordova-plugin-camera/pull/311#issuecomment-425912543
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Windows phone correct filename
> --
>
> Key: CB-13836
> URL: https://issues.apache.org/jira/browse/CB-13836
> Project: Apache Cordova
>  Issue Type: Improvement
>  Components: cordova-plugin-camera
>Reporter: Mark de Groot
>Priority: Minor
>
> All photos are stored as 'photo.jpg' of 'photo.png'. Change the filename to 
> match Android behaviour. 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13849) getDirectory('') returns root path

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13849?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634042#comment-16634042
 ] 

ASF GitHub Bot commented on CB-13849:
-

janpio commented on issue #230: CB-13849: (android) getDirectory("") returns 
current path instead of / path
URL: 
https://github.com/apache/cordova-plugin-file/pull/230#issuecomment-425912496
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> getDirectory('') returns root path
> --
>
> Key: CB-13849
> URL: https://issues.apache.org/jira/browse/CB-13849
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Affects Versions: cordova-android-7.0.0
> Environment: Android
>Reporter: Tobias Haupenthal
>Priority: Major
>
> At the moment the command \{someDir}.getDirectory("") with a blank argument 
> returns the root-path.
> If someone works already on a subdirectory that behaviour might be a security 
> flaw.
> Better would be to return the current directory.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13774) cordova-plugin-media broken recording and playback functionality

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634037#comment-16634037
 ] 

ASF GitHub Bot commented on CB-13774:
-

janpio commented on issue #164: CB-13774: repair broken recording and playback 
functionality on Android
URL: 
https://github.com/apache/cordova-plugin-media/pull/164#issuecomment-425912074
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> cordova-plugin-media broken recording and playback functionality 
> -
>
> Key: CB-13774
> URL: https://issues.apache.org/jira/browse/CB-13774
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-media
> Environment: Android
>Reporter: Mark Mitchell
>Priority: Blocker
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> The problem seems to be that somehow the encoder and decoder are set to 
> different encoding types and are not compatible. Additionally its assuming an 
> encoder is available on the platform and since the interface to the this 
> object does not allow a way to specifying encoding I feel like the default 
> may have been more appropriate. 
> Currently the code is as follows 
> {code:java}
> this.recorder.setOutputFormat(MediaRecorder.OutputFormat.AAC_ADTS); // 
> RAW_AMR);
> this.recorder.setAudioEncoder(MediaRecorder.AudioEncoder.AAC); //AMR_NB);
> {code}
> But trying to play back the recorded audio fails with a decoder error



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13633) Report info/warning events on Android

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13633?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634036#comment-16634036
 ] 

ASF GitHub Bot commented on CB-13633:
-

janpio commented on issue #156: CB-13633: (android) Propagate buffering 
start/stop events on Android.
URL: 
https://github.com/apache/cordova-plugin-media/pull/156#issuecomment-425911924
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Report info/warning events on Android
> -
>
> Key: CB-13633
> URL: https://issues.apache.org/jira/browse/CB-13633
> Project: Apache Cordova
>  Issue Type: Improvement
>  Components: cordova-plugin-media
>Reporter: Michael Goffioul
>Priority: Minor
>
> In order to get notified of buffering events, it is handy to report 
> info/warning events (MediaPlayer.OnInfoListener) up to the web application.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-6938) iOS File plugin: copyTo for file returns error when destination file exists

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-6938?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634038#comment-16634038
 ] 

ASF GitHub Bot commented on CB-6938:


janpio commented on issue #211: CB-6938: (fix) on iOS, copyTo should overwrite 
any existing file
URL: 
https://github.com/apache/cordova-plugin-file/pull/211#issuecomment-425912239
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> iOS File plugin: copyTo for file returns error when destination file exists
> ---
>
> Key: CB-6938
> URL: https://issues.apache.org/jira/browse/CB-6938
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-plugin-file
>Affects Versions: 3.5.0
> Environment: ios
>Reporter: vldm
>Assignee: Ian Clelland
>Priority: Major
>  Labels: iOS
>
> According to w3c spec:
> A copy of a file on top of an existing file must attempt to delete and 
> replace that file.
> iOS version of plugin in file CDVLocalFilesystem.m contains this code for the 
> case:
> {code}
> } else if (bNewExists) {
> // the full destination should NOT already exist if a copy
> errCode = PATH_EXISTS_ERR;
> {code}
> Suggested patch:
> {code}
> diff --git a/src/ios/CDVLocalFilesystem.m b/src/ios/CDVLocalFilesystem.m
> index d1576ad..f5aa508 100644
> --- a/src/ios/CDVLocalFilesystem.m
> +++ b/src/ios/CDVLocalFilesystem.m
> @@ -535,11 +535,11 @@
>  if (bSrcIsDir && ![self canCopyMoveSrc:srcFullPath 
> ToDestination:newFileSystemPath]) {
>  // can't copy dir into self
>  errCode = INVALID_MODIFICATION_ERR;
> -} else if (bNewExists) {
> -// the full destination should NOT already exist if a 
> copy
> -errCode = PATH_EXISTS_ERR;
>  } else {
> -bSuccess = [fileMgr copyItemAtPath:srcFullPath 
> toPath:newFileSystemPath error:];
> +   bSuccess = YES;
> +if (bNewExists)
> +bSuccess = [fileMgr 
> removeItemAtPath:newFileSystemPath error:];
> +if (bSuccess)
> +bSuccess = [fileMgr copyItemAtPath:srcFullPath 
> toPath:newFileSystemPath error:];
>  }
>  } else { // move
>  // iOS requires that destination must not exist before 
> calling moveTo
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13852) cordova-plugin-media getPosition audio failed

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13852?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634035#comment-16634035
 ] 

ASF GitHub Bot commented on CB-13852:
-

janpio commented on issue #165: CB-13852: (iOS) Fix bug prevent error on 
getPosition
URL: 
https://github.com/apache/cordova-plugin-media/pull/165#issuecomment-425911880
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> cordova-plugin-media getPosition audio failed
> -
>
> Key: CB-13852
> URL: https://issues.apache.org/jira/browse/CB-13852
> Project: Apache Cordova
>  Issue Type: Improvement
>  Components: cordova-plugin-media
>Affects Versions: cordova-ios 4.5.0
> Environment: iOS all versions
>Reporter: Juan Sebastian Dussan Cubides
>Priority: Minor
>
> (iOS)
> When I try to get the position of an audio that had an error while loading, 
> the crash appose exponentially because it can not cast the time = none



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13632) UDP not handled as streaming on Android

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634033#comment-16634033
 ] 

ASF GitHub Bot commented on CB-13632:
-

janpio commented on issue #155: CB-13632: (android) Allow UDP media streaming.
URL: 
https://github.com/apache/cordova-plugin-media/pull/155#issuecomment-425911778
 
 
   Hey, I just fixed the problem that caused Android tests to fail in `master`. 
Could you rebase this PR please? This should get rid of the Android failures 
and possibly fix all test failures for this PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> UDP not handled as streaming on Android
> ---
>
> Key: CB-13632
> URL: https://issues.apache.org/jira/browse/CB-13632
> Project: Apache Cordova
>  Issue Type: Improvement
>  Components: cordova-plugin-media
>Reporter: Michael Goffioul
>Priority: Minor
>
> When using a UDP multicast URL as media source, it is not handled as a 
> streaming content by AudioPlayer.isStreaming().



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13460) Apply Android style to the splash screen.

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13460?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16634027#comment-16634027
 ] 

ASF GitHub Bot commented on CB-13460:
-

janpio commented on issue #137: CB-13460: Apply Android style to the splash 
screen
URL: 
https://github.com/apache/cordova-plugin-splashscreen/pull/137#issuecomment-425911395
 
 
   Hey, I just fixed the problem that caused Android tests to fail. Rebasing on 
master should get rid of those and possibly remove all test failures for this 
PR.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Apply Android style to the splash screen.
> -
>
> Key: CB-13460
> URL: https://issues.apache.org/jira/browse/CB-13460
> Project: Apache Cordova
>  Issue Type: Improvement
>  Components: cordova-plugin-splashscreen
>Reporter: Piotr Dyraga
>Priority: Major
>
> This improvement aims at making the splash screen dialog use the 
> `SplashScreenStyle`, if defined. If not, it plugin should fall back to the 
> default style (backward compatibility).



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-7179) [InAppBrowser][iOS 8] Update to support WKWebView

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-7179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16633973#comment-16633973
 ] 

ASF GitHub Bot commented on CB-7179:


janpio edited a comment on issue #271: CB-7179 (iOS): Add WKWebView support for 
iOS
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/271#issuecomment-425896071
 
 
   The test failure was caused by a timeout - the test seemed to need a tiny 
bit more than the 5 minutes that were configured as timeout. As I was working 
on similar problems with Android, I just doubled that for now to 10 minutes. 
Didn't help a thing.
   
   Closer look at the tests on Saucelabs show that there is some code looping: 
https://saucelabs.com/beta/tests/04cc3e87441e4252af89a7587e240d73/commands#12 
No idea what and why (yet).
   
   But:
   You could rebase (hopefully) one last time onto `master`, I have re-enabled 
another iOS test on Travis that was disabled for no good reason and most 
importantly fixed the unrelated Android test(s) that are currently failing here 
as well. At least those should be fixed then.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [InAppBrowser][iOS 8] Update to support WKWebView
> -
>
> Key: CB-7179
> URL: https://issues.apache.org/jira/browse/CB-7179
> Project: Apache Cordova
>  Issue Type: Sub-task
>  Components: cordova-plugin-inappbrowser
> Environment: iOS 8
>Reporter: Shazron Abdullah
>Priority: Major
>
> support dual use with UIWebView



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-7179) [InAppBrowser][iOS 8] Update to support WKWebView

2018-10-01 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-7179?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16633954#comment-16633954
 ] 

ASF GitHub Bot commented on CB-7179:


janpio commented on issue #271: CB-7179 (iOS): Add WKWebView support for iOS
URL: 
https://github.com/apache/cordova-plugin-inappbrowser/pull/271#issuecomment-425896071
 
 
   The test failure was caused by a timeout - the test seemed to need a tiny 
bit more than the 5 minutes that were configured as timeout. As I was working 
on similar problems with Android, I just doubled that for now to 10 minutes. 
Didn't help a thing.
   
   Closer look at the tests on Saucelabs show that there is some code looping: 
https://saucelabs.com/beta/tests/04cc3e87441e4252af89a7587e240d73/commands#12 
No idea what and why (yet).
   
   But:
   You could rebase (hopefully) one last time onto `master`, I have re-enabled 
another iOS test on Travis that was disabled for no good reason and most 
importantly fixed the unrelated Android test(s) that are failing here as well.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [InAppBrowser][iOS 8] Update to support WKWebView
> -
>
> Key: CB-7179
> URL: https://issues.apache.org/jira/browse/CB-7179
> Project: Apache Cordova
>  Issue Type: Sub-task
>  Components: cordova-plugin-inappbrowser
> Environment: iOS 8
>Reporter: Shazron Abdullah
>Priority: Major
>
> support dual use with UIWebView



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-14166) Cordova on windows fails when adding plugin

2018-10-01 Thread ASF subversion and git services (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-14166?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16633873#comment-16633873
 ] 

ASF subversion and git services commented on CB-14166:
--

Commit 7ac19a8b086e8fa19551cbfc2e188c3bd7072da1 in cordova-lib's branch 
refs/heads/cross-spawn-test from Almir Kadric
[ https://gitbox.apache.org/repos/asf?p=cordova-lib.git;h=7ac19a8 ]

CB-14166: (cli) Fixed issue when install plugins on windows

The npm package name is already wrapped in quotes to prevent special
characters from being execute. However the additional win32 code was
further injecting quotes around the version causing npm to return a
EINVALIDTAGNAME error. The win32 code is redundant and seems to not
be required.


> Cordova on windows fails when adding plugin
> ---
>
> Key: CB-14166
> URL: https://issues.apache.org/jira/browse/CB-14166
> Project: Apache Cordova
>  Issue Type: Bug
>  Components: cordova-windows
>Reporter: takuya
>Assignee: Jesse MacFadyen
>Priority: Blocker
> Attachments: screenshot-1.png, スクリーンショット 2018-09-06 11.36.42.png
>
>
> cordova on windows fails to add the plugin.
> This error happens even when cordova-plugin-whitelist.
> Therefore `cordova platform add windows` also fails.
> For example,
> ```
> > cordova create sample
> > cd sample
> > cordova platform add windows
> ```
> brings following error.
> ```
> ...
> Check your connection and plugin name/version/URL.
> Error: C:\Program Files (x86)\Nodist\bin\npm.exe: Command failed with exit 
> code 1 Error output:
> npm ERR! code EINVALIDTAGNAME
> npm ERR! Invalid tag name ""1"": Tags may not have any characters that 
> encodeURIComponent encodes.
> ```
> This error happens for windows environment only.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org



[jira] [Commented] (CB-13745) Add build-tools-26.0.2 to travis for Android tests

2018-10-01 Thread Jan Piotrowski (Sujan) (JIRA)


[ 
https://issues.apache.org/jira/browse/CB-13745?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16633856#comment-16633856
 ] 

Jan Piotrowski (Sujan) commented on CB-13745:
-

Did you do this manually or do you have a script to make as many changes across 
as many repos?

> Add build-tools-26.0.2 to travis for Android tests
> --
>
> Key: CB-13745
> URL: https://issues.apache.org/jira/browse/CB-13745
> Project: Apache Cordova
>  Issue Type: Bug
>Reporter: jcesarmobile
>Priority: Major
>
> Android tests are failing in plugins because of unaccepted licenses, to fix 
> it we have to add build-tools-26.0.2 to the travis.yml for all plugins



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org