fix(credentials): check expiry properly (#190)

grasevski · web-flow · commit 89e4c0bfe1f0 · 2025-02-10T17:46:13.000-08:00
Expiry flag was inverted when checking credentials. This causes ChainedCredentialsProvider to fail.
diff --git a/include/miniocpp/credentials.h b/include/miniocpp/credentials.h
@@ -69,7 +69,7 @@ struct Credentials {
   bool IsExpired() const { return expired(expiration); }
 
   explicit operator bool() const {
-    return !err && !access_key.empty() && expired(expiration);
+    return !err && !access_key.empty() && !expired(expiration);
   }
 
   static Credentials ParseXML(std::string_view data, const std::string& root);

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,7 @@ struct Credentials {`
`69`	`69`	`bool IsExpired() const { return expired(expiration); }`
`70`	`70`
`71`	`71`	`explicit operator bool() const {`
`72`		`- return !err && !access_key.empty() && expired(expiration);`
	`72`	`+ return !err && !access_key.empty() && !expired(expiration);`
`73`	`73`	`}`
`74`	`74`
`75`	`75`	`static Credentials ParseXML(std::string_view data, const std::string& root);`