[WPT Export] Remove all newlines in GitHub auth header

third_party/WebKit/Tools/Scripts/webkitpy/w3c/wpt_github.py

 # Copyright 2016 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.
 
 import base64
 import json
 import logging
 import urllib2
 
 
 _log = logging.getLogger(__name__)
 API_BASE = 'https://api.github.com'
 EXPORT_LABEL = 'chromium-export'
 
 
 class WPTGitHub(object):
 
     def __init__(self, host, user, token):
         self.host = host
         self.user = user
         self.token = token
         assert self.user and self.token
 
     def auth_token(self):
-        return base64.encodestring('{}:{}'.format(self.user, self.token)).strip()
+        return base64.encodestring('{}:{}'.format(self.user, self.token)).replace('\n', '')

[qyearsley, 2017/02/13 23:29:12]

Or, as just discussed: base64.b64encode('{}:{}'.format(self.user, self.token))

 
     def request(self, path, method, body=None):
         assert path.startswith('/')
         if body:
             body = json.dumps(body)
         opener = urllib2.build_opener(urllib2.HTTPHandler)
         request = urllib2.Request(url=API_BASE + path, data=body)
         request.add_header('Accept', 'application/vnd.github.v3+json')
         request.add_header('Authorization', 'Basic {}'.format(self.auth_token()))
         request.get_method = lambda: method
@@ skipping 60 matching lines @@
     def delete_remote_branch(self, remote_branch_name):
         # TODO(jeffcarp): Unit test this method
         path = '/repos/w3c/web-platform-tests/git/refs/heads/%s' % remote_branch_name
         data, status_code = self.request(path, method='DELETE')
 
         if status_code != 204:
             # TODO(jeffcarp): Raise more specific exception (create MergeError class?)
             raise Exception('Received non-204 status code attempting to delete remote branch: {}'.format(status_code))
 
         return data