X-Git-Url: http://nitlanguage.org

diff --git a/lib/github/github_curl.nit b/lib/github/github_curl.nit
index 5f6339f..6f28546 100644
--- a/lib/github/github_curl.nit
+++ b/lib/github/github_curl.nit
@@ -18,15 +18,18 @@ module github_curl
 
 import curl
 import json::static
+import json
 
 # Specific Curl that know hot to talk to the github API
 class GithubCurl
-	super Curl
 
 	# Headers to use on all requests
 	var header: HeaderMap is noinit
 
 	# OAuth token
+	#
+	# Use an empty string to disable authentication and connect
+	# anonymously (thus less capabilities and more rate limits)
 	var auth: String
 
 	# User agent (is used by github to contact devs in case of problems)
@@ -35,14 +38,14 @@ class GithubCurl
 
 	init do
 		header = new HeaderMap
-		header["Authorization"] = "token {auth}"
+		if auth != "" then header["Authorization"] = "token {auth}"
 	end
 
 	# Get the requested URI, and check the HTTP response. Then convert to JSON
 	# and check for Github errors.
-	fun get_and_check(uri: String): nullable Jsonable
+	fun get_and_check(uri: String): nullable Serializable
 	do
-		var request = new CurlHTTPRequest(uri, self)
+		var request = new CurlHTTPRequest(uri)
 		request.user_agent = user_agent
 		request.headers = header
 		var response = request.execute
@@ -50,7 +53,7 @@ class GithubCurl
 		if response isa CurlResponseSuccess then
 			var obj = response.body_str.parse_json
 			if obj isa JsonObject then
-				if obj.keys.has("message") then
+				if obj.keys.has("message") and obj.keys.has("documentation_url") then
 					print "Message from Github API: {obj["message"] or else ""}"
 					print "Requested URI: {uri}"
 					abort
@@ -71,18 +74,18 @@ class GithubCurl
 	# Then convert to JSON and check for Github errors.
 	# Unlike `get_and_check`, error do not trigger an abort but
 	# are reported as `GithubError`.
-	fun get_and_parse(uri: String): nullable Jsonable
+	fun get_and_parse(uri: String): nullable Serializable
 	do
-		var request = new CurlHTTPRequest(uri, self)
+		var request = new CurlHTTPRequest(uri)
 		request.user_agent = user_agent
 		request.headers = header
 		var response = request.execute
 		if response isa CurlResponseSuccess then
 			var obj = response.body_str.parse_json
 			if obj isa JsonObject then
-				if obj.keys.has("message") then
+				if obj.keys.has("message") and obj.keys.has("documentation_url") then
 					var title = "GithubAPIError"
-					var msg = obj["message"].to_s or else ""
+					var msg = obj["message"].as(not null).to_s
 					var err = new GithubError(msg, title)
 					err.json["requested_uri"] = uri
 					err.json["status_code"] = response.status_code
@@ -111,7 +114,7 @@ end
 # Check the `json` value to find them.
 class GithubError
 	super Error
-	super Jsonable
+	super Serializable
 
 	# The name of the error.
 	var name: String
@@ -125,18 +128,18 @@ class GithubError
 		json["message"] = message.to_json
 	end
 
-	redef fun to_json do
-		return json.to_json
-	end
+	redef fun serialize_to(v) do json.serialize_to v
 
 	redef fun to_s do return "[{name}] {super}"
 end
 
+# Gets the Github token from `git` configuration
+#
 # Return the value of `git config --get github.oauthtoken`
-# return "" if no such a key
+# or `""` if no key exists.
 fun get_github_oauth: String
 do
-	var p = new IProcess("git", "config", "--get", "github.oauthtoken")
+	var p = new ProcessReader("git", "config", "--get", "github.oauthtoken")
 	var token = p.read_line
 	p.wait
 	p.close