organicmaps/organicmaps-tmp
Archived
13
Fork 1
Code Issues 2.7k Pull requests 182 Projects Releases 65 Wiki Activity Actions

[ios] Fixed crash on urls with spaces #621

Merged
biodranik merged 1 commit from ios/deeplink-space-crash-fix into master 2021-06-18 12:02:07 +00:00
Conversation 0 Commits 1 Files changed 1 +3 -1
1 changed files with 3 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
iphone/Maps/Core/DeepLink/DeepLinkHandler.swift
View file

@ -28,7 +28,9 @@
func applicationDidReceiveUniversalLink(_ universalLink: URL) -> Bool {
// Convert http(s)://omaps.app/ENCODEDCOORDS/NAME to om://ENCODEDCOORDS/NAME
self.url = URL(string: String(format: "om:/%@", universalLink.path))
self.url = URL(string: universalLink.absoluteString
.replacingOccurrences(of: "http://omaps.app", with: "om:/")
.replacingOccurrences(of: "https://omaps.app", with: "om:/"))
return handleDeepLink(url: self.url!)
vng commented 2021-06-18 11:24:11 +00:00 (Migrated from github.com)
Review
Copy link

А давай сделаем безопасно и проверим тут self.url ?
Нам подсунут говноссылку и мы так будем падать

А давай сделаем безопасно и проверим тут self.url ? Нам подсунут говноссылку и мы так будем падать
vng commented 2021-06-18 11:41:52 +00:00 (Migrated from github.com)
Review
Copy link

Ок, в принципе если входной URL валидный, то тут тоже будет ок.

Ок, в принципе если входной URL валидный, то тут тоже будет ок.
}