refactored to NewProvider

main.go

@@ -46,9 +46,9 @@ func main() {
 		case "seafile":
 			sharecmd.provider = seafile.NewProvider(sharecmd.config.ProviderSettings["url"], sharecmd.config.ProviderSettings["token"], sharecmd.config.ProviderSettings["repoid"])
 		case "googledrive":
-			sharecmd.provider = googledrive.NewGoogleDriveProvider(sharecmd.config.ProviderSettings["googletoken"])
+			sharecmd.provider = googledrive.NewProvider(sharecmd.config.ProviderSettings["googletoken"])
 		case "dropbox":
-			sharecmd.provider = dropbox.NewDropboxProvider(cfg.ProviderSettings["token"])
+			sharecmd.provider = dropbox.NewProvider(cfg.ProviderSettings["token"])
 		default:
 			config.Setup(*configFile)
 			os.Exit(0)

provider/dropbox/dropbox.go

@@ -26,9 +26,10 @@ var (
 	personalAppSecret = "mi55rqbz0rgb16f"
 )
 
-// Tokenmap example: { "token": "xxx" }
+// TokenMap example: { "token": "xxx" }
 type TokenMap map[string]string
 
+// OAuth2DropboxConfig creates a oauth config
 func OAuth2DropboxConfig() *oauth2.Config {
 	return &oauth2.Config{
 		ClientID:     personalAppKey,
@@ -70,23 +71,24 @@ func writeTokens(filePath string, tokens TokenMap) {
 	}
 }
 
-// DropboxProvider implements a provider using dropbox sdk
-type DropboxProvider struct {
+// Provider implements a provider using dropbox sdk
+type Provider struct {
 	Config dropbox.Config
 	token  string
 }
 
-func NewDropboxProvider(token string) *DropboxProvider {
-
-	return &DropboxProvider{
+// NewProvider creates a new Provider
+func NewProvider(token string) *Provider {
+	return &Provider{
 		Config: dropbox.Config{
 			Token:    token,
 			LogLevel: dropbox.LogOff,
 		},
 	}
 }
 
-func (c *DropboxProvider) Upload(file *os.File, path string) (dst string, err error) {
+// Upload the file to dropbox
+func (c *Provider) Upload(file *os.File, path string) (dst string, err error) {
 	//
 	if path == "" {
 		path = "/"
@@ -126,7 +128,8 @@ func (c *DropboxProvider) Upload(file *os.File, path string) (dst string, err er
 	return dst, nil
 }
 
-func (c *DropboxProvider) GetLink(filepath string) (string, error) {
+// GetLink for file
+func (c *Provider) GetLink(filepath string) (string, error) {
 	share := sharing.New(c.Config)
 	arg := sharing.NewCreateSharedLinkWithSettingsArg(filepath)
 

provider/googledrive/googledrive.go

@@ -9,18 +9,17 @@ import (
 	"os"
 	"path/filepath"
 
-	humanize "github.com/dustin/go-humanize"
-	//"github.com/mitchellh/ioprogress"
 	"github.com/coreos/ioprogress"
+	humanize "github.com/dustin/go-humanize"
 	"golang.org/x/oauth2"
 	"golang.org/x/oauth2/google"
 	drive "google.golang.org/api/drive/v3"
 )
 
 //https://developers.google.com/drive/api/v3/quickstart/go
 
-// GoogleDriveProvider implements a provider
-type GoogleDriveProvider struct {
+// Provider implements a provider
+type Provider struct {
 	Config *oauth2.Config
 	token  *oauth2.Token
 }
@@ -36,21 +35,23 @@ func OAuth2GoogleDriveConfig() *oauth2.Config {
 	return config
 }
 
-func NewGoogleDriveProvider(token string) *GoogleDriveProvider {
+// NewProvider creates a new Provider
+func NewProvider(token string) *Provider {
 	tok := &oauth2.Token{}
 	err := json.Unmarshal([]byte(token), tok)
 	if err != nil {
 		log.Fatalf("Unable to parse config file: %v", err)
 	}
 
-	return &GoogleDriveProvider{token: tok, Config: OAuth2GoogleDriveConfig()}
+	return &Provider{token: tok, Config: OAuth2GoogleDriveConfig()}
 }
 
-func (c *GoogleDriveProvider) getClient() *http.Client {
+func (c *Provider) getClient() *http.Client {
 	return c.Config.Client(context.Background(), c.token)
 }
 
-func (c *GoogleDriveProvider) Upload(file *os.File, path string) (fileID string, err error) {
+// Upload the file
+func (c *Provider) Upload(file *os.File, path string) (fileID string, err error) {
 
 	fileInfo, err := file.Stat()
 	if err != nil {
@@ -84,7 +85,8 @@ func (c *GoogleDriveProvider) Upload(file *os.File, path string) (fileID string,
 	return r.Id, nil
 }
 
-func (c *GoogleDriveProvider) GetLink(filepath string) (string, error) {
+// GetLink for fileid
+func (c *Provider) GetLink(filepath string) (string, error) {
 	fileID := filepath
 
 	client := c.getClient()