Update the appengine code to use OAuth2 and break the gdata dependency.

remoting/client/appengine/api.py

 #!/usr/bin/env python
 
 # Copyright (c) 2011 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """API endpoints to get around javascript's single-origin restriction."""
 
 import logging
 
 from django.utils import simplejson as json
 
-import gdata.client
-
+from google.appengine.api import urlfetch
 from google.appengine.ext import webapp
 from google.appengine.ext.webapp import util
 from google.appengine.ext.webapp.util import login_required
 import auth
 
 
 class GetXmppTokenHandler(webapp.RequestHandler):
   """Retrieves the user's XMPP token."""
   @login_required
   def get(self):
     try:
       self.response.headers['Content-Type'] = 'application/json'
       self.response.out.write(
           json.dumps({'xmpp_token': auth.GetXmppToken().token}))
     except auth.NotAuthenticated:
       self.response.out.write('User has not authenticated')
       self.set_status(400)
 
+
 class GetHostListHandler(webapp.RequestHandler):
   """Proxies the host-list handlers on the Chromoting directory."""
   @login_required
   def get(self):
-    try:
-      client = gdata.client.GDClient()
-      host_list_json = client.Request(
-          method='GET',
-          uri="https://www.googleapis.com/chromoting/v1/@me/hosts",
-          converter=None,
-          desired_class=None,
-          auth_token=auth.GetChromotingToken())
-      self.response.headers['Content-Type'] = 'application/json'
-      self.response.out.write(host_list_json.read())
-    except auth.NotAuthenticated:
-      self.response.out.write('User has not authenticated')
-      self.response.set_status(400)
-    except (gdata.client.Unauthorized, gdata.client.RequestError), inst:
-      self.response.out.write(inst.reason)
-      self.response.set_status(inst.status)
+    result = urlfetch.fetch(
+        url = 'https://www.googleapis.com/chromoting/v1/@me/hosts',
+        method = urlfetch.GET,
+        headers = {'Authorization': 'OAuth ' + auth.GetAccessToken()})
+    self.response.headers['Content-Type'] = 'application/json'

[Jamie, 2011/05/20 16:46:47]

In the previous code, the content-type was only set on success. Does this
matter?

[awong, 2011/05/20 20:37:38]

Good catch.  Actually, I'm just relaying the full set of headers from the
googleapi's response below, so this is redundant since they should be setting
the Content-Type, and we should just be a passthru.

+    self.response.out.write(result.content)

[Jamie, 2011/05/20 16:46:47]

Does |write| write the data to the client immediately? If so then don't we need
to set the status and headers before doing that?

[awong, 2011/05/20 20:37:38]

I'm pretty sure it's not a streaming response.  But just to be sure, I
reordered.

+    for i in result.headers:
+      self.response.headers[i] = result.headers[i]
+    self.response.set_status(result.status_code)
+
 
 def main():
   application = webapp.WSGIApplication(
       [
       ('/api/get_xmpp_token', GetXmppTokenHandler),
       ('/api/get_host_list', GetHostListHandler)
       ],
       debug=True)
   util.run_wsgi_app(application)
 
 
 if __name__ == '__main__':
   main()