fix token cache TTL

The TTL was supplied to the middleware as a duration and then in that middleware multiplied by `time.Second` again. Durations should not be multiplied because they result in unintended values.
```go
	time.Second * 1 = 1s
	time.Second * time.Second = 277777h46m40s
```
This commit is contained in:
David Christofas
2021-02-22 18:37:14 +01:00
parent e535975718
commit 7ad38d7757
2 changed files with 6 additions and 2 deletions

View File

@@ -0,0 +1,5 @@
Bugfix: Fix the ttl of the authentication middleware cache
The authentication cache ttl was multiplied with `time.Second` multiple times. This resulted in a ttl that was not intended.
https://github.com/owncloud/ocis/pull/1699

View File

@@ -5,7 +5,6 @@ import (
"net/http"
"regexp"
"strings"
"time"
)
var (
@@ -114,7 +113,7 @@ func newOIDCAuth(options Options) func(http.Handler) http.Handler {
HTTPClient(options.HTTPClient),
OIDCIss(options.OIDCIss),
TokenCacheSize(options.UserinfoCacheSize),
TokenCacheTTL(time.Second*time.Duration(options.UserinfoCacheTTL)),
TokenCacheTTL(options.UserinfoCacheTTL),
CredentialsByUserAgent(options.CredentialsByUserAgent),
)
}