search

nordnet / cordova-universal-links-plugin

[DEPRECATED] - Cordova plugin to support Universal/Deep Links for iOS/Android.
https://github.com/nordnet/cordova-universal-links-plugin/issues/160
MIT License
349 stars 529 forks source link

Issues when working with URL's containing special characters #100

Closed chriswmfa closed 6 years ago

chriswmfa commented 7 years ago

Hi All,

Has anybody ran into any issues when using special characters in URL's? I am attempting to put the following into the path url: /link?url=/Work/ does not appear to work at all where as /Work/Test/ does work fine.

Thanks

nikDemyankov commented 7 years ago

Hi,

Can you, please, share:

chriswmfa commented 7 years ago

Hi,

I am testing this on iOS currently, the url's I am using are:

<universal-links>
    <host name="xxx.com" >
        <path url="*/link?url=/MyPortal/Authenticate/*" event="openLink" />
        <path url="*/link?url=/MyPortal/Documents/*" event="openLink" />
        <path url="*/MyPortal/Authenticate/*" event="openLink" />
        <path url="*/MyPortal/Documents/*" event="openLink" />
    </host>
</universal-links>

When I go to https://xxx.com/MyPortal/Authenticate/ or https://xxx.com/MyPortal/Documents/

Then it works as it should, but if i go to https://xxx.com/link?url=/MyPortal/Authenticate/ it doesn't appear to work.

Thanks,

nikDemyankov commented 7 years ago

Yes, it doesn't work. You can't use query parameters for url matching. They are ignored by the operating system.

chriswmfa commented 7 years ago

Is there any way around this? Or is this just something that I have to work around?

nikDemyankov commented 7 years ago

This story is similar to https://github.com/nordnet/cordova-universal-links-plugin/issues/86 . You can check it out.

In short - you can capture all links from your website by defining only <host name="xxx.com"/>, and then process paths with query params on the JS side to detect, which view to present to the user.

nordnet-deprecation-bot commented 6 years ago

👋 Hi! Thank you for your interest in this repo.

đŸ˜ĸ We are not using nordnet/cordova-universal-links-plugin anymore, and we lack the manpower and the experience needed to maintain it. We are aware of the inconveniece that this may cause you. Feel free to use it as is, or create your own fork.

🔒 This will now be closed & locked.

ℹī¸ Please see #160 for more information.