Skip to content
This repository has been archived by the owner on Oct 1, 2018. It is now read-only.

Array Parameters Not Parsed Properly #55

Closed
izhekov opened this issue Mar 23, 2016 · 3 comments
Closed

Array Parameters Not Parsed Properly #55

izhekov opened this issue Mar 23, 2016 · 3 comments
Labels

Comments

@izhekov
Copy link

izhekov commented Mar 23, 2016

URLs containing array parameters like "dcid[]=1&dcid[]=2&dcid[]=3" are not parsed properly under Android. Those are missing from the event's data "params" property. I did not tested the same under iOS.

@nikDemyankov
Copy link
Member

Okey, thanks for the info. I think you can get the actual URL from the url parameter of the event data, and then read these params from it.

@izhekov
Copy link
Author

izhekov commented Mar 23, 2016

That's exactly what I'm doing. I'm manually parsing the needed parameters at the JS side. I can see the issue is probably related of using the Uri's getQueryParameterNames(). In fact the method probably works just fine as http://www.faqs.org/rfcs/rfc2396.html says those are not allowed:

Other characters are excluded because gateways and other transport agents are known to sometimes modify such characters, or they are used as delimiters.
unwise = "{" | "}" | "|" | "" | "^" | "[" | "]" | "`"

Still I believe this pattern is quite commonly used and it will be good if it's supported by your plugin.

@nikDemyankov nikDemyankov added this to the v1.2.1 milestone Sep 8, 2016
@nikDemyankov nikDemyankov removed this from the v1.2.1 milestone Oct 23, 2016
@nordnet-deprecation-bot
Copy link
Contributor

👋 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.

@nordnet nordnet locked and limited conversation to collaborators Sep 19, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

3 participants