From 8aa7c87835fafb65e1be4f70f92425b13d1d6d0d Mon Sep 17 00:00:00 2001 From: Teemu Patja Date: Wed, 8 Mar 2017 22:06:15 +0200 Subject: [PATCH] Remove "repo" oauth scope * remove "repo" from required permissions (fixes: #29) * formatting --- src/clj/commiteth/github/core.clj | 2 +- test/clj/commiteth/test/webhooks.clj | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/clj/commiteth/github/core.clj b/src/clj/commiteth/github/core.clj index 8b08743..5aadb33 100644 --- a/src/clj/commiteth/github/core.clj +++ b/src/clj/commiteth/github/core.clj @@ -27,7 +27,7 @@ (defn authorize-url [] (let [params (codec/form-encode {:client_id (client-id) :redirect_uri (redirect-uri) - :scope "admin:repo_hook user:email repo admin:org_hook" + :scope "admin:repo_hook user:email admin:org_hook" :allow_signup true :state (str (UUID/randomUUID))})] (str "https://github.com/login/oauth/authorize" "?" params))) diff --git a/test/clj/commiteth/test/webhooks.clj b/test/clj/commiteth/test/webhooks.clj index 3a9088b..5395877 100644 --- a/test/clj/commiteth/test/webhooks.clj +++ b/test/clj/commiteth/test/webhooks.clj @@ -18,5 +18,4 @@ fixes #123 [comment]: # (To auto-close issue on merge, please insert the related issue number after # i.e fixes #566) "] - (is (= '(123) (webhooks/extract-issue-number body title)))) - )) + (is (= '(123) (webhooks/extract-issue-number body title))))))