diff options
author | Alex Mayer <amayer5125@gmail.com> | 2019-06-20 00:26:02 -0400 |
---|---|---|
committer | Alex Mayer <amayer5125@gmail.com> | 2019-06-20 00:26:02 -0400 |
commit | ab116f62282cb201ba2903d24bbce9eb03ee3e38 (patch) | |
tree | cec4eb17a532b60ee9b6cbadefda68b1a45362ba | |
parent | 376f08237158c7ceab15cec4b9e8af5ca0d293a1 (diff) | |
download | gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.tar gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.tar.gz gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.tar.bz2 gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.tar.lz gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.tar.xz gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.tar.zst gh_authkey_checker-ab116f62282cb201ba2903d24bbce9eb03ee3e38.zip |
Consolidate User Exists Check Into fetchKeys()
-rw-r--r-- | main.go | 25 |
1 files changed, 4 insertions, 21 deletions
@@ -25,6 +25,10 @@ func fetchKeys(username string) (string, error) { } defer resp.Body.Close() + if resp.StatusCode == http.StatusNotFound { + return "", fmt.Errorf("%s is an invalid user", username) + } + if resp.StatusCode != http.StatusOK { return "", fmt.Errorf("Expected http 200 but got %d instead", resp.StatusCode) } @@ -37,33 +41,12 @@ func fetchKeys(username string) (string, error) { return string(bodyBytes), nil } -func checkUsername(username string) error { - url := fmt.Sprintf("https://github.com/%s.keys", username) - response, err := http.Get(url) - if err != nil { - return err - } - - if response.StatusCode != http.StatusOK { - return fmt.Errorf("%s is an invalid user", username) - } - - return nil -} - func main() { username, err := parseArgs() if err != nil { log.Fatal(err) } - log.Printf("Checking for GitHub user %s", username) - err = checkUsername(username) - if err != nil { - log.Fatal(err) - } - log.Printf("Found valid user %s", username) - log.Printf("Fetching keys for user %s", username) keys, err := fetchKeys(username) if err != nil { |