fix: deep link null param skip (#1859)

* Made deep link reading more resilient
decentraland · Aug 26, 2024 · 6da76bc · 6da76bc
1 parent cc9aa48
commit 6da76bc
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/Explorer/Assets/Scripts/Global/AppArgs/ApplicationParametersParser.cs b/Explorer/Assets/Scripts/Global/AppArgs/ApplicationParametersParser.cs
@@ -84,7 +84,11 @@ private void ProcessDeepLinkParameters(string deepLinkString)
             NameValueCollection uriQuery = HttpUtility.ParseQueryString(uri.Query);
 
             foreach (string uriQueryKey in uriQuery.AllKeys)
+            {
+                // if the deep link is not constructed correctly (AKA 'decentraland://?&blabla=blabla') a 'null' parameter can be detected...
+                if (uriQueryKey == null) continue;
                 appParameters[uriQueryKey] = uriQuery.Get(uriQueryKey);
+            }
 
             // Patch for WinOS sometimes affecting the 'realm' parameter in deep links putting a '/' at the end
             if (appParameters.TryGetValue(REALM_PARAM, out string? realmParamValue) && realmParamValue.EndsWith('/'))