Skip to content
This repository
Browse code

IsItUp.com doesn't actually support unicode either

disabled the feature and tests for now :-/
  • Loading branch information...
commit d22da1b6351b28daf68b8ac25e4438c89df86216 1 parent 54eb70a
Dylan Lloyd authored March 21, 2013
2  lib/DDG/Spice/IsItUp.pm
@@ -15,7 +15,7 @@ status "enabled";
15 15
 
16 16
 spice is_cached => 0;
17 17
 
18  
-triggers query_lc => qr,^((?:is\s|))(?:http://)?([[:alnum:]\-]+(?:\.[[:alnum:]\-]+)*?)(?:(\.[[:alpha:]]{2\,4})|)\s(?:up|down|working),;
  18
+triggers query_lc => qr/^((?:is\s|))(?:http:\/\/)?([0-9a-z\-]+(?:\.[0-9a-z\-]+)*?)(?:(\.[a-z]{2,4})|)\s(?:up|down|working)/i;
19 19
 
20 20
 spice to => 'http://isitup.org/$1.json?callback={{callback}}';
21 21
 
13  t/IsItUp.t
@@ -14,18 +14,15 @@ ddg_spice_test(
14 14
         caller => 'DDG::Spice::IsItUp',
15 15
         is_cached => 0
16 16
     ),
17  
-    'is føtex.dk up?' => test_spice(
18  
-        '/js/spice/is_it_up/f%C3%B8tex.dk',
19  
-        call_type => 'include',
20  
-        caller => 'DDG::Spice::IsItUp',
21  
-        is_cached => 0
22  
-    ),
23  
-    'is http://føtex.dk up?' => test_spice(
24  
-        '/js/spice/is_it_up/f%C3%B8tex.dk',
  17
+    'is http://duckduckgo.com up?' => test_spice(
  18
+        '/js/spice/is_it_up/duckduckgo.com',
25 19
         call_type => 'include',
26 20
         caller => 'DDG::Spice::IsItUp',
27 21
         is_cached => 0
28 22
     ),
  23
+    # unfortunately, the isitup.com api does not
  24
+    # currently support unicode or ssl
  25
+    'is føtex.dk up?' => undef,
29 26
     'is https://føtex.dk up?' => undef,
30 27
 );
31 28
 

0 notes on commit d22da1b

Please sign in to comment.
Something went wrong with that request. Please try again.