status-im/open-bounty
2
0
Fork 0
mirror of https://github.com/status-im/open-bounty.git synced 2025-02-20 13:18:22 +00:00
Code Issues Projects Releases Wiki Activity

Remove "repo" oauth scope

Browse Source 
* remove "repo" from required permissions (fixes: #29)
* formatting
This commit is contained in:
Teemu Patja 2017-03-08 22:06:15 +02:00
parent 6e60802803
commit 8aa7c87835
No known key found for this signature in database
GPG Key ID: F5B7035E6580FD4C
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/clj/commiteth/github/core.clj
View File

@ -27,7 +27,7 @@
(defn authorize-url [] (defn authorize-url []
(let [params (codec/form-encode {:client_id (client-id) (let [params (codec/form-encode {:client_id (client-id)
:redirect_uri (redirect-uri) :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 :allow_signup true
:state (str (UUID/randomUUID))})] :state (str (UUID/randomUUID))})]
(str "https://github.com/login/oauth/authorize" "?" params))) (str "https://github.com/login/oauth/authorize" "?" params)))

3
test/clj/commiteth/test/webhooks.clj
View File

@ -18,5 +18,4 @@
fixes #123 fixes #123
[comment]: # (To auto-close issue on merge, please insert the related issue number after # i.e fixes #566) [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))))))
))