diff options
| -rw-r--r-- | .travis.yml | 6 | ||||
| -rw-r--r-- | server/auth/github/github.go | 6 | ||||
| -rw-r--r-- | server/fs/s3.go | 1 | ||||
| -rw-r--r-- | templates/token.go | 1 |
4 files changed, 10 insertions, 4 deletions
diff --git a/.travis.yml b/.travis.yml index dc81a35..3e2fdac 100644 --- a/.travis.yml +++ b/.travis.yml @@ -6,7 +6,10 @@ env: go: - 1.5.4 - 1.6.2 - - tip +# - tip + +before_install: + - go get -v github.com/golang/lint/golint sudo: false script: @@ -14,3 +17,4 @@ script: - go list ./... |grep -v vendor/ |xargs go test - gofmt -d $(find -type f -name '*.go' -not -path './vendor/*') - go list ./... |grep -v vendor/ |xargs go vet + - go list ./... |grep -v vendor/ |xargs -L1 golint diff --git a/server/auth/github/github.go b/server/auth/github/github.go index a6a4a59..912caae 100644 --- a/server/auth/github/github.go +++ b/server/auth/github/github.go @@ -76,9 +76,9 @@ func (c *Config) Valid(token *oauth2.Token) bool { return member } -// GitHub doesn't seem to allow token revocation - tokens are indefinite and -// there are no refresh options etc. Returns nil to satisfy the Provider -// interface. +// Revoke is a no-op revoke method. GitHub doesn't seem to allow token +// revocation - tokens are indefinite and there are no refresh options etc. +// Returns nil to satisfy the Provider interface. func (c *Config) Revoke(token *oauth2.Token) error { return nil } diff --git a/server/fs/s3.go b/server/fs/s3.go index 4e82bb4..e16e7d6 100644 --- a/server/fs/s3.go +++ b/server/fs/s3.go @@ -19,6 +19,7 @@ import ( "github.com/nsheridan/cashier/server/config" ) +// Register the /s3/ filesystem as a well-known filesystem. func Register(config *config.AWS) { ac := &aws.Config{} // If region is unset the SDK will attempt to read the region from the environment. diff --git a/templates/token.go b/templates/token.go index dbad371..270055b 100644 --- a/templates/token.go +++ b/templates/token.go @@ -1,5 +1,6 @@ package templates +// Token is the page users see when authenticated. const Token = `<html> <head> <title>YOUR TOKEN!</title> |