You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cordova.apache.org by GitBox <gi...@apache.org> on 2021/04/24 15:37:24 UTC
[GitHub] [cordova-android] breautek opened a new issue #1217: Application does not initialize correctly on nightly/dev master
breautek opened a new issue #1217:
URL: https://github.com/apache/cordova-android/issues/1217
# Bug Report
## Problem
### What is expected to happen?
Application to start correctly
### What does actually happen?
Application fails to initialize probably (deviceReady not fired).
## Information
<!-- Include all relevant information that might help understand and reproduce the problem -->
This is a regression from a newly added feature https://github.com/apache/cordova-android/pull/1137
With the default configuration, plugins will fail to initialise because they do not pass the whitelist checks. This occurs when you have at least one plugin installed.
The default `<content src="index.html" />` seems to produce a white list pattern where `index.html` is the `host`, instead of the `path`, so when comparing against the `https://localhost` origin, it returns false.
### Command or Code
<!-- What command or code is needed to reproduce the problem? -->
```
cordova create test
cd test
cordova platform add android@nightly
cordova plugin add cordova-plugin-file
cordova run android
```
These steps will produce the hello world cordova app that stays on "connecting" because `deviceReady` never fires.
### Environment, Platform, Device
<!-- In what environment, on what platform or on which device are you experiencing the issue? -->
`cordova-android@nightly`
And one other plugin
### Version information
<!--
What are relevant versions you are using?
For example:
Cordova: Cordova CLI, Cordova Platforms, Cordova Plugins
Other Frameworks: Ionic Framework and CLI version
Operating System, Android Studio, Xcode etc.
-->
## Checklist
<!-- Please check the boxes by putting an x in the [ ] like so: [x] -->
- [x] I searched for existing GitHub issues
- [x] I updated all Cordova tooling to most recent version
- [x] I included all the necessary information above
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-android] erisu closed issue #1217: Application does not initialize correctly on 10.x nightly/dev master
Posted by GitBox <gi...@apache.org>.
erisu closed issue #1217:
URL: https://github.com/apache/cordova-android/issues/1217
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-android] jcesarmobile commented on issue #1217: Application does not initialize correctly on 10.x nightly/dev master
Posted by GitBox <gi...@apache.org>.
jcesarmobile commented on issue #1217:
URL: https://github.com/apache/cordova-android/issues/1217#issuecomment-838737646
It also works if you add `<allow-navigation href="https://localhost/*" />`, but yeah, it should work without the need of adding it or changing the content url.
I think the proper place to put it is inside the AllowList plugin, once it gets merged https://github.com/apache/cordova-android/pull/1138
there is the check that allows the content src url
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org
[GitHub] [cordova-android] NiklasMerz commented on issue #1217: Application does not initialize correctly on 10.x nightly/dev master
Posted by GitBox <gi...@apache.org>.
NiklasMerz commented on issue #1217:
URL: https://github.com/apache/cordova-android/issues/1217#issuecomment-879122505
Looks like we need to change this: https://github.com/apache/cordova-android/blob/53d60dd70719220a655b1bedb09b20280d92b0d3/framework/src/org/apache/cordova/AllowListPlugin.java#L81
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@cordova.apache.org
For additional commands, e-mail: issues-help@cordova.apache.org