[BUG] Crashes app when app name contains spaces
Closed this issue · 2 comments
chronikum commented
Hello, during development I found this issue
![image](https://private-user-images.githubusercontent.com/34622984/255634593-96105bba-d71e-4ac6-8ef0-3bbc4d5008ac.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNzY3MzcsIm5iZiI6MTcyMDI3NjQzNywicGF0aCI6Ii8zNDYyMjk4NC8yNTU2MzQ1OTMtOTYxMDViYmEtZDcxZS00YWM2LThlZjAtM2JiYzRkNTAwOGFjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA2VDE0MzM1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWM3MmNmYTMwZDFkMzcxZGY1ZDEzYjVkZjQ2NTAwOWFkYmU5NGYzMDRlMTQ0NDQ2Mjg0ZjlmZTU4MDEzZjcxYTUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.2EtUcQIB5pwJzKmgrrF-XCpwsgm8Vpl5dhkZPC7zszQ)
It seems like, looking at the frame variables that appName seems to cause the the issue
![image](https://private-user-images.githubusercontent.com/34622984/255634712-f82b6012-0c5e-4868-a0b5-e4d9c372cc52.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjAyNzY3MzcsIm5iZiI6MTcyMDI3NjQzNywicGF0aCI6Ii8zNDYyMjk4NC8yNTU2MzQ3MTItZjgyYjYwMTItMGM1ZS00ODY4LWEwYjUtZTRkOWMzNzJjYzUyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MDYlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzA2VDE0MzM1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTY2Mjg5MDQzOGZlYjNjMGIwODI1YWQxY2ZjYjllODM4MDE5N2FmN2JmNDM1OWVjMjhjNGUwZTJjN2Y4ZjFhNTkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.B6w57FmkFWW8Jy6gbu86CwpqwXqGdGBqVNApgPlxGkk)
because the url cannot be constructed, as there is a space in the app name.
Of course, this would be valid for any character that is not allowed in urls.
Several apps have this issue when looking at the default list:
omit2c commented
It's also not necessary for the AppStore link to have the app name in it.
The link will also work like this: https://apps.apple.com/app/id\(appID)
hiddevdploeg commented
Should be fixed with #21