Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(284)

Unified Diff: common/auth/internal/iam.go

Issue 2951393002: [errors] de-specialize Transient in favor of Tags. (Closed)
Patch Set: copyright Created 3 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: common/auth/internal/iam.go
diff --git a/common/auth/internal/iam.go b/common/auth/internal/iam.go
index 3aebdc6f930bb1f5d9d8ef76722a5b41d11acbfd..6b7c79e204f125c2d9f448b3ba47367507714511 100644
--- a/common/auth/internal/iam.go
+++ b/common/auth/internal/iam.go
@@ -12,9 +12,9 @@ import (
"golang.org/x/oauth2"
"google.golang.org/api/googleapi"
- "github.com/luci/luci-go/common/errors"
"github.com/luci/luci-go/common/gcloud/googleoauth"
"github.com/luci/luci-go/common/gcloud/iam"
+ "github.com/luci/luci-go/common/retry"
)
type iamTokenProvider struct {
@@ -70,7 +70,7 @@ func (p *iamTokenProvider) MintToken(ctx context.Context, base *oauth2.Token) (*
if apiErr, _ := err.(*googleapi.Error); apiErr != nil && apiErr.Code < 500 {
return nil, err
}
- return nil, errors.WrapTransient(err)
+ return nil, retry.Tag.Apply(err)
}
func (p *iamTokenProvider) RefreshToken(ctx context.Context, prev, base *oauth2.Token) (*oauth2.Token, error) {

Powered by Google App Engine
This is Rietveld 408576698