Open theigl opened 3 weeks ago
This would probably be fixed by passing all URLs in allowNavigation
instead of the server.url
as suggested in #7454.
This issue has been labeled as type: bug
. This label is added to issues that that have been reproduced and are being tracked in our internal issue tracker.
Capacitor Version
:pill: Capacitor Doctor :pill: Latest Dependencies: @capacitor/cli: 6.1.0 @capacitor/core: 6.1.0 @capacitor/android: 6.1.0 @capacitor/ios: 6.1.0 Installed Dependencies: @capacitor/cli: 6.1.0 @capacitor/android: 6.1.0 @capacitor/core: 6.1.0 @capacitor/ios: 6.1.0 [success] Android looking great! :ok_hand:
Other API Details
No response
Platforms Affected
Current Behavior
JS does not get injected into the page when
appUrl
contains a query parameter.The Android Bridge attempts to normalize the path before passing it as
allowedOrigins
toWebViewCompat.addDocumentStartJavaScript
, but it does not strip query parameters:https://github.com/ionic-team/capacitor/blob/0ad0c3f23e1c62ade5081878e345e74f963b9ea5/android/capacitor/src/main/java/com/getcapacitor/Bridge.java#L257-L266
Android includes the query parameter when evaluating
allowedOrigins
and JS does not get injected into the DOM.Expected Behavior
JS gets injected into the page when the
appUrl
contains query parameters.Project Reproduction
https://github.com/divatzk/capacitor-server-url-query-parameter/tree/main/capacitor-app
Additional Information
The issue can be seen in the sample project where server.url contains a query parameter:
Run the app as is and you should get this result:
If the query parameter is removed, Capacitor JS gets injected successfully: