Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Cyrillic and Arabic characters in API 4 #148

Closed
andrieslouw opened this issue Jul 31, 2018 · 2 comments
Closed

Cyrillic and Arabic characters in API 4 #148

andrieslouw opened this issue Jul 31, 2018 · 2 comments
Assignees
@kleisauke
Labels
bug Something isn't working completed Feature or request has been completed
Milestone
v4.0.0

Comments

@andrieslouw
Copy link
Member

andrieslouw commented Jul 31, 2018 •
edited
Loading

Seems like the implementation of Cyrillic and Arabic characters in API 4 is broken.

Test-cases:
https://images.weserv.nl/?url=кто.рф/images/logo_rf.png

https://images.weserv.nl/?url=thumbs.жить.рф/unsafe/351x351/smart/https://xn--f1ae4a2b.xn--p1ai/media/work/content/8AAGQy3aEu.jpg

https://images.weserv.nl/?url=жить.рф/media/work/content/8AAGQy3aEu.jpg
@andrieslouw andrieslouw added bug Something isn't working triage This issue is being investigated labels Jul 31, 2018
@andrieslouw andrieslouw added this to the v4.0.0 milestone Jul 31, 2018
@andrieslouw
Copy link
Member Author

#13

@andrieslouw
Copy link
Member Author

This test-case is working, seems to be only the punycode-hostname is causing problems:
https://images.weserv.nl/?url=www.almega.com.hk/testing/測試圖片.jpg

@andrieslouw andrieslouw added completed Feature or request has been completed and removed triage This issue is being investigated labels Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kleisauke kleisauke

Labels
bug Something isn't working completed Feature or request has been completed
Milestone
v4.0.0
Development

No branches or pull requests
2 participants
@andrieslouw @kleisauke