fix: provider may not even suppport signout

adhocore · adhocore · commit 25dde2a430e5 · 2021-10-15T14:54:04.000+07:00
diff --git a/goic.go b/goic.go
@@ -433,19 +433,19 @@ func (g *Goic) SignOut(tok *Token, redir string, res http.ResponseWriter, req *h
 		}
 	}
 
-	name := tok.Provider
-	if !g.Supports(name) {
+	p, ok := g.providers[tok.Provider]
+	if !ok || p.wellKnown.SignOutURI == "" {
 		return ErrProviderSupport
 	}
 
 	tk := tok.AccessToken
 	if tk == "" && tok.RefreshToken != "" {
 		tk = tok.RefreshToken
-	} else {
+	}
+	if tk == "" {
 		return ErrTokenAccessKey
 	}
 
-	p := g.providers[tok.Provider]
 	redirect, err := http.NewRequest("GET", p.wellKnown.SignOutURI, nil)
 	if err != nil {
 		return err

Original file line number	Diff line number	Diff line change
`@@ -433,19 +433,19 @@ func (g Goic) SignOut(tok Token, redir string, res http.ResponseWriter, req *h`
`433`	`433`	`}`
`434`	`434`	`}`
`435`	`435`
`436`		`- name := tok.Provider`
`437`		`- if !g.Supports(name) {`
	`436`	`+ p, ok := g.providers[tok.Provider]`
	`437`	`+ if !ok \|\| p.wellKnown.SignOutURI == "" {`
`438`	`438`	`return ErrProviderSupport`
`439`	`439`	`}`
`440`	`440`
`441`	`441`	`tk := tok.AccessToken`
`442`	`442`	`if tk == "" && tok.RefreshToken != "" {`
`443`	`443`	`tk = tok.RefreshToken`
`444`		`- } else {`
	`444`	`+ }`
	`445`	`+ if tk == "" {`
`445`	`446`	`return ErrTokenAccessKey`
`446`	`447`	`}`
`447`	`448`
`448`		`- p := g.providers[tok.Provider]`
`449`	`449`	`redirect, err := http.NewRequest("GET", p.wellKnown.SignOutURI, nil)`
`450`	`450`	`if err != nil {`
`451`	`451`	`return err`