github: remove unused functions

I doubt anyone is using this client, and if they are, they can switch.

Author: kevinburke1

github-release.go

@@ -8,8 +8,6 @@ import (
 	"github.com/voxelbrain/goptions"
 )
 
-const GH_URL = "https://github.com"
-
 type Options struct {
 	Help      goptions.Help `goptions:"-h, --help, description='Show this help'"`
 	Verbosity []bool        `goptions:"-v, --verbose, description='Be verbose'"`

github/file.go

@@ -1,10 +1,7 @@
 package github
 
 import (
-	"bytes"
-	"errors"
 	"fmt"
-	"io"
 	"os"
 )
 
@@ -44,36 +41,3 @@ func fsizeSeek(f *os.File) (int64, error) {
 	}
 	return off, nil
 }
-
-// materializeFile takes a physical file or stream (named pipe, user input,
-// ...) and returns an io.Reader and the number of bytes that can be read
-// from it.
-func materializeFile(f *os.File) (io.Reader, int64, error) {
-	fi, err := f.Stat()
-	if err != nil {
-		return nil, 0, err
-	}
-
-	// If the file is actually a char device (like user typed input)
-	// or a named pipe (like a streamed in file), buffer it up.
-	//
-	// When uploading a file, you need to either explicitly set the
-	// Content-Length header or send a chunked request. Since the
-	// github upload server doesn't accept chunked encoding, we have
-	// to set the size of the file manually. Since a stream doesn't have a
-	// predefined length, it's read entirely into a byte buffer.
-	if fi.Mode()&(os.ModeCharDevice|os.ModeNamedPipe) == 1 {
-		vprintln("input was a stream, buffering up")
-
-		var buf bytes.Buffer
-		n, err := buf.ReadFrom(f)
-		if err != nil {
-			return nil, 0, errors.New("req: could not buffer up input stream: " + err.Error())
-		}
-		return &buf, n, err
-	}
-
-	// We know the os.File is most likely an actual file now.
-	n, err := GetFileSize(f)
-	return f, n, err
-}

github/github.go

@@ -19,24 +19,6 @@ const DefaultBaseURL = "https://api.github.com"
 // Set to values > 0 to control verbosity, for debugging.
 var VERBOSITY = 0
 
-// DoAuthRequest ...
-//
-// TODO: This function is amazingly ugly (separate headers, token, no API
-// URL constructions, et cetera).
-func DoAuthRequest(method, url, mime, token string, headers map[string]string, body io.Reader) (*http.Response, error) {
-	req, err := newAuthRequest(method, url, mime, token, headers, body)
-	if err != nil {
-		return nil, err
-	}
-
-	resp, err := http.DefaultClient.Do(req)
-	if err != nil {
-		return nil, err
-	}
-
-	return resp, nil
-}
-
 // Client collects a few options that can be set when contacting the GitHub
 // API, such as authorization tokens. Methods called on Client will supply
 // these options when calling the API.
@@ -227,45 +209,6 @@ func (c Client) getPaginated(uri string) (io.ReadCloser, error) {
 	return r, nil
 }
 
-// Create a new request that sends the auth token.
-func newAuthRequest(method, url, mime, token string, headers map[string]string, body io.Reader) (*http.Request, error) {
-	vprintln("creating request:", method, url, mime, token)
-
-	var n int64 // content length
-	var err error
-	if f, ok := body.(*os.File); ok {
-		// Retrieve the content-length and buffer up if necessary.
-		body, n, err = materializeFile(f)
-		if err != nil {
-			return nil, err
-		}
-	}
-
-	req, err := http.NewRequest(method, url, body)
-	if err != nil {
-		return nil, err
-	}
-
-	// net/http automatically does this if req.Body is of type
-	// (bytes.Reader|bytes.Buffer|strings.Reader). Sadly, we also need to
-	// handle *os.File.
-	if n != 0 {
-		vprintln("setting content-length to", n)
-		req.ContentLength = n
-	}
-
-	if mime != "" {
-		req.Header.Set("Content-Type", mime)
-	}
-	req.Header.Set("Authorization", "token "+token)
-
-	for k, v := range headers {
-		req.Header.Set(k, v)
-	}
-
-	return req, nil
-}
-
 // nextLink returns the HTTP header Link annotated with 'next', "" otherwise.
 func nextLink(links linkheader.Links) string {
 	for _, link := range links {