{"action":"created","issue":{"url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/issues/220","repository_url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice","labels_url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/issues/220/labels{/name}","comments_url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/issues/220/comments","events_url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/issues/220/events","html_url":"https://github.com/duckduckgo/zeroclickinfo-spice/issues/220","id":20069971,"number":220,"title":"IsItUp: Add support for domains containing unicode characters","user":{"login":"moollaza","id":873785,"avatar_url":"https://avatars.githubusercontent.com/u/873785?v=3","gravatar_id":"","url":"https://api.github.com/users/moollaza","html_url":"https://github.com/moollaza","followers_url":"https://api.github.com/users/moollaza/followers","following_url":"https://api.github.com/users/moollaza/following{/other_user}","gists_url":"https://api.github.com/users/moollaza/gists{/gist_id}","starred_url":"https://api.github.com/users/moollaza/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/moollaza/subscriptions","organizations_url":"https://api.github.com/users/moollaza/orgs","repos_url":"https://api.github.com/users/moollaza/repos","events_url":"https://api.github.com/users/moollaza/events{/privacy}","received_events_url":"https://api.github.com/users/moollaza/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/labels/Improvement","name":"Improvement","color":"d7e102"},{"url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/labels/Low-Hanging%20Fruit","name":"Low-Hanging Fruit","color":"009800"},{"url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/labels/Priority:%20Medium","name":"Priority: Medium","color":"eb6420"},{"url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/labels/Triggering","name":"Triggering","color":"f7c6c7"}],"state":"open","locked":false,"assignee":{"login":"MrChrisW","id":5282264,"avatar_url":"https://avatars.githubusercontent.com/u/5282264?v=3","gravatar_id":"","url":"https://api.github.com/users/MrChrisW","html_url":"https://github.com/MrChrisW","followers_url":"https://api.github.com/users/MrChrisW/followers","following_url":"https://api.github.com/users/MrChrisW/following{/other_user}","gists_url":"https://api.github.com/users/MrChrisW/gists{/gist_id}","starred_url":"https://api.github.com/users/MrChrisW/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MrChrisW/subscriptions","organizations_url":"https://api.github.com/users/MrChrisW/orgs","repos_url":"https://api.github.com/users/MrChrisW/repos","events_url":"https://api.github.com/users/MrChrisW/events{/privacy}","received_events_url":"https://api.github.com/users/MrChrisW/received_events","type":"User","site_admin":false},"milestone":null,"comments":22,"created_at":"2013-09-25T21:05:06Z","updated_at":"2016-06-16T04:26:45Z","closed_at":null,"body":"Eg. [\"is føtex.dk up\"](https://duckduckgo.com/?q=is+f%C3%B8tex.dk+up)\r\n\r\n- \"expand føtex.dk\"\r\n- \"whois føtex.dk\".\r\n\r\nSeems that their API might not support unicode. Needs further investigation!\r\n\r\nhttps://duck.co/ia/view/is_it_up"},"comment":{"url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/issues/comments/226385358","html_url":"https://github.com/duckduckgo/zeroclickinfo-spice/issues/220#issuecomment-226385358","issue_url":"https://api.github.com/repos/duckduckgo/zeroclickinfo-spice/issues/220","id":226385358,"user":{"login":"rasikapohankar","id":8025692,"avatar_url":"https://avatars.githubusercontent.com/u/8025692?v=3","gravatar_id":"","url":"https://api.github.com/users/rasikapohankar","html_url":"https://github.com/rasikapohankar","followers_url":"https://api.github.com/users/rasikapohankar/followers","following_url":"https://api.github.com/users/rasikapohankar/following{/other_user}","gists_url":"https://api.github.com/users/rasikapohankar/gists{/gist_id}","starred_url":"https://api.github.com/users/rasikapohankar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rasikapohankar/subscriptions","organizations_url":"https://api.github.com/users/rasikapohankar/orgs","repos_url":"https://api.github.com/users/rasikapohankar/repos","events_url":"https://api.github.com/users/rasikapohankar/events{/privacy}","received_events_url":"https://api.github.com/users/rasikapohankar/received_events","type":"User","site_admin":false},"created_at":"2016-06-16T04:26:45Z","updated_at":"2016-06-16T04:26:45Z","body":"@moollaza Yup, using `domain_to_ascii` from Net::IDN::Encode on the query works with PublicSuffix. I was happy and about to make a pull request when I realised it works only if the tld does not have unicode characters. The `tld_tree` that Domain::PublicSuffix makes from public_suffix_list.dat has names as `\\x{...}` and so if we do domain to ascii, the tld as 'xn--' is not found. I was trying to search if there was anything more, will see Net::Domain::TLD.\r\nIf we remove the regex trigger and make it a word we'd still need to do these operations on the remainder, no?"}}
{"actor":{"display_login":"rasikapohankar"}}