[Bug]: CapacitorHttp removing query params when using native fetch

[marcelobotega]

Capacitor Version

💊 Capacitor Doctor 💊

Latest Dependencies:

@capacitor/cli: 5.7.2
@capacitor/core: 5.7.2
@capacitor/android: 5.7.2
@capacitor/ios: 5.7.2

Installed Dependencies:

@capacitor/ios: not installed
@capacitor/cli: 5.7.2
@capacitor/android: 5.7.2
@capacitor/core: 5.7.2

[success] Android looking great! 👌

Other API Details

❯ npm --version
10.2.4

❯ pnpm --version
8.15.1

❯ node --version
v20.11.1

Platforms Affected

• iOS
• Android
• Web

Current Behavior

Since @capacitor/[email protected] when calling an API using the browser fetch with CapacitorHttp enabled, the interceptor is removing the query params.
CapacitorHttp.get works normally.

Expected Behavior

The query params should not be removed from the URL

Project Reproduction

https://github.com/marcelobotega/capacitor-http-error

Additional Information

The instructions for the reproduction repo are on the readme.
The problem do not happen on @capacitor/[email protected] you can downgrade the version on package.json and the API calls will have the query params.
The placeholder API will work, but the ?postId=2 params will not be on the URL on version >= 5.7.1

[colbymelvin]

Seeing this also (on iOS and Android), I believe it was introduced here.

Specifically, that change appears to be constructing a proxy URL in order to send a request through the appropriate CapacitorHttp interceptor. However, it's only using the pathname property from the original URL object to construct the proxy URL. Notably pathname will exclude an existing query string.

[booleanbetrayal]

This seems like a major regression that could lead to lots of application breakage.

[patrickjquinn]

This has entirely broken my project as I'm sure it has for many. This should be considered a sev1 regression IMO. It also impacts iOS too FYI.

[ionitron-bot]

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.

[ciekawy]

I did try 5.7.3 and now I'm getting Failed to load resource: Could not connect to the server.

and the url instead of
https://<host>/<path>?<queryparams>
is:
https://localhost/_capacitor_https_interceptor_/<host>/<path>?<queryparams>

[ionitron-bot]

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Capacitor, please create a new issue and ensure the template is fully filled out.