onedrive: use token renewer to stop auth errors on long uploads

Fixes #820
This commit is contained in:
Nick Craig-Wood 2017-01-29 20:42:43 +00:00
parent bd29015022
commit 74702554da

View File

@ -80,12 +80,13 @@ func init() {
// Fs represents a remote one drive // Fs represents a remote one drive
type Fs struct { type Fs struct {
name string // name of this remote name string // name of this remote
root string // the path we are working on root string // the path we are working on
features *fs.Features // optional features features *fs.Features // optional features
srv *rest.Client // the connection to the one drive server srv *rest.Client // the connection to the one drive server
dirCache *dircache.DirCache // Map of directory path to directory id dirCache *dircache.DirCache // Map of directory path to directory id
pacer *pacer.Pacer // pacer for API calls pacer *pacer.Pacer // pacer for API calls
tokenRenewer *oauthutil.Renew // renew the token on expiry
} }
// Object describes a one drive object // Object describes a one drive object
@ -179,7 +180,7 @@ func errorHandler(resp *http.Response) error {
// NewFs constructs an Fs from the path, container:path // NewFs constructs an Fs from the path, container:path
func NewFs(name, root string) (fs.Fs, error) { func NewFs(name, root string) (fs.Fs, error) {
root = parsePath(root) root = parsePath(root)
oAuthClient, _, err := oauthutil.NewClient(name, oauthConfig) oAuthClient, ts, err := oauthutil.NewClient(name, oauthConfig)
if err != nil { if err != nil {
log.Fatalf("Failed to configure One Drive: %v", err) log.Fatalf("Failed to configure One Drive: %v", err)
} }
@ -199,6 +200,12 @@ func NewFs(name, root string) (fs.Fs, error) {
return nil, errors.Wrap(err, "failed to get root") return nil, errors.Wrap(err, "failed to get root")
} }
// Renew the token in the background
f.tokenRenewer = oauthutil.NewRenew(f.String(), ts, func() error {
_, _, err := f.readMetaDataForPath("")
return err
})
f.dirCache = dircache.New(root, rootInfo.ID, f) f.dirCache = dircache.New(root, rootInfo.ID, f)
// Find the current root // Find the current root
@ -924,6 +931,9 @@ func (o *Object) uploadMultipart(in io.Reader, size int64) (err error) {
// //
// The new object may have been created if an error is returned // The new object may have been created if an error is returned
func (o *Object) Update(in io.Reader, src fs.ObjectInfo) (err error) { func (o *Object) Update(in io.Reader, src fs.ObjectInfo) (err error) {
o.fs.tokenRenewer.Start()
defer o.fs.tokenRenewer.Stop()
size := src.Size() size := src.Size()
modTime := src.ModTime() modTime := src.ModTime()