Updated config check.
This commit is contained in:
+2
-3
@@ -32,15 +32,14 @@ function validateConfig(config) {
|
||||
|
||||
const hasApiKeys = Array.isArray(config.apiKeys) && config.apiKeys.length > 0;
|
||||
const hasSvcAccounts = Array.isArray(config.serviceAccounts) && config.serviceAccounts.length > 0;
|
||||
const hasCredPath = !!config.credentialsPath;
|
||||
|
||||
// Do not allow both API keys and service accounts to be present simultaneously
|
||||
if (hasApiKeys && hasSvcAccounts) {
|
||||
throw new Error('Configuration must not include both `apiKeys` and `serviceAccounts`; choose one authentication method');
|
||||
}
|
||||
|
||||
if (!hasApiKeys && !hasSvcAccounts && !hasCredPath) {
|
||||
throw new Error('Configuration must include at least one of: `apiKeys`, `serviceAccounts`, or `credentialsPath`');
|
||||
if (!hasApiKeys && !hasSvcAccounts) {
|
||||
throw new Error('Configuration must include at least one `apiKeys` or `serviceAccounts`');
|
||||
}
|
||||
|
||||
// Basic validation of documents/sheets structure
|
||||
|
||||
Reference in New Issue
Block a user