From: Ralf Jung <post@ralfj.de>
Date: Thu, 3 Jun 2021 15:33:12 +0000 (+0200)
Subject: Merge pull request #3 from damien-list/master
X-Git-Url: https://git.ralfj.de/git-mirror.git/commitdiff_plain/4a11da9e0c8aa9ddfdb519dccef1d5dac4ed3bd8?hp=f1fabccc7a0449afc66b20176a05eada3a5dd581

Merge pull request #3 from damien-list/master

change deprecated usage of access_token in the url to use instead authorization header.
---

diff --git a/github-add-hooks.py b/github-add-hooks.py
index 3a2d66c..22d0d0f 100755
--- a/github-add-hooks.py
+++ b/github-add-hooks.py
@@ -15,14 +15,14 @@ def generate_ssh_key(name, bits):
     subprocess.check_call(["ssh-keygen", "-f", name, "-C", name, "-b", str(bits), "-q", "-N", ""])
 
 def add_deploy_key(key_name, repo_owner, repo_name, access_token):
-    url = "https://api.github.com/repos/{owner}/{repo}/keys?access_token={token}".format(owner=repo_owner, repo=repo_name, token=access_token)
+    url = "https://api.github.com/repos/{owner}/{repo}/keys".format(owner=repo_owner, repo=repo_name)
     data = { 'title': os.path.basename(key_name), 'key': open(key_name+".pub").read() }
-    r = requests.post(url, data=json.dumps(data))
+    r = requests.post(url, data=json.dumps(data), headers = {"Authorization": "token {token}".format(token=access_token)})
     if r.status_code >= 300:
         raise Exception(str(json.loads(r.content.decode('utf-8'))))
     
 def add_web_hook(webhook_url, hmac_secret, repo_owner, repo_name, access_token):
-    url = 'https://api.github.com/repos/{owner}/{repo}/hooks?access_token={token}'.format(owner=repo_owner, repo=repo_name, token=access_token)
+    url = 'https://api.github.com/repos/{owner}/{repo}/hooks'.format(owner=repo_owner, repo=repo_name)
     data = {
         'name': "web",
         'active': True,
@@ -33,7 +33,7 @@ def add_web_hook(webhook_url, hmac_secret, repo_owner, repo_name, access_token):
             'secret': hmac_secret,
         }
     }
-    r = requests.post(url, data=json.dumps(data))
+    r = requests.post(url, data=json.dumps(data), headers = {"Authorization": "token {token}".format(token=access_token)})
     if r.status_code >= 300:
         raise Exception(str(json.loads(r.content.decode('utf-8'))))