Merge pull request #809 from drdamour/sosearch

fixed sosearch to not use wwwdude as mentioned in #700

Author: Tom Bell

src/scripts/sosearch.coffee

@@ -2,7 +2,7 @@
 #   Search stack overflow and provide links to the first 5 questions
 #
 # Dependencies:
-#   "wwwdude": "0.1.0"
+#   none
 #
 # Configuration:
 #   None
@@ -13,8 +13,10 @@
 #
 # Author:
 #   carsonmcdonald
+#   drdamour
 
-wwwdude = require("wwwdude")
+
+zlib = require('zlib');
 
 # API keys are public for Stackapps
 hubot_stackapps_apikey = 'BeOjD228tEOZP6gbYoChsg'
@@ -23,39 +25,50 @@ module.exports = (robot) ->
   robot.respond /sosearch( me)? (.*)/i, (msg) ->
     re = RegExp("(.*) with tags (.*)", "i")
     opts = msg.match[2].match(re)
+
     if opts?
-      soSearch msg, escape(opts[1]), opts[2].split(',')
+      soSearch msg, opts[1], opts[2].split(',')
     else 
-      soSearch msg, escape(msg.match[2]), null
+      soSearch msg, msg.match[2], []
 
 soSearch = (msg, search, tags) ->
-  client = wwwdude.createClient
-    headers: { 'User-Agent': 'hubot search' }
-    gzip: true
-    timeout: 500
-
-  tagged = ''
-  if tags?
-    for tag in tags
-      tagged += "#{tag.replace(/^\s+|\s+$/g,'')};"
-    tagged = "&tagged=#{escape(tagged[0..tagged.length-2])}"
+
+  
+  
+  data = ""
+  msg.http("http://api.stackoverflow.com/1.1/search")
+    .query
+      intitle: encodeURIComponent(search)
+      key: hubot_stackapps_apikey
+      tagged: encodeURIComponent(tags.join(':'))
+    .get( (err, req)->
+      req.addListener "response", (res)->
+        output = res
+        
+        #pattern stolen from http://stackoverflow.com/questions/10207762/how-to-use-request-or-http-module-to-read-gzip-page-into-a-string
+        if res.headers['content-encoding'] is 'gzip'
+          output = zlib.createGunzip()
+          res.pipe(output)
+
+        output.on 'data', (d)->
+          data += d.toString('utf-8')
+
+        output.on 'end', ()->
+          parsedData = JSON.parse(data)
+          if parsedData.error
+            msg.send "Error searching stack overflow: #{parsedData.error.message}"
+            return
+
+          if parsedData.total > 0
+            qs = for question in parsedData.questions[0..5]
+              "http://www.stackoverflow.com/questions/#{question.question_id} - #{question.title}"
+            if parsedData.total-5 > 0
+              qs.push "#{parsedData.total-5} more..."
+            for ans in qs
+              msg.send ans
+          else
+            msg.reply "No questions found matching that search."
+    )()
+
+
 
-  client.get("http://api.stackoverflow.com/1.1/search?intitle=#{search}#{tagged}&key=#{hubot_stackapps_apikey}")
-  .addListener 'error', (err) ->
-    console.log("Error: #{err}")
-    msg.reply "Error while executing search."
-  .addListener 'http-error', (data, resp) ->
-    console.log("Error code: #{resp.statusCode}")
-    msg.reply "Error while executing search."
-  .addListener 'success', (data, resp) ->
-    results = JSON.parse(data)
-
-    if results.total > 0
-      qs = for question in results.questions[0..5]
-        "http://www.stackoverflow.com/questions/#{question.question_id} - #{question.title}"
-      if results.total-5 > 0
-        qs.push "#{results.total-5} more..."
-      for ans in qs
-        msg.send ans
-    else
-      msg.reply "No questions found matching that search."