Skip to content

Commit

Permalink
exclude CA from auto-ops service setup config
Browse files Browse the repository at this point in the history
follow up of elastic#200073
  • Loading branch information
ashokaditya committed Nov 14, 2024
1 parent 86b3738 commit c6cddb7
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
1 change: 0 additions & 1 deletion x-pack/plugins/data_usage/server/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ export const configSchema = schema.object({
schema.object({
certificate: schema.maybe(schema.string()),
key: schema.maybe(schema.string()),
ca: schema.maybe(schema.string()),
})
),
})
Expand Down
5 changes: 1 addition & 4 deletions x-pack/plugins/data_usage/server/services/autoops_api.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,7 @@ export class AutoOpsAPIService {
logger.debug(
`[AutoOps API] Creating autoops agent with TLS cert: ${
autoopsConfig?.api?.tls?.certificate ? '[REDACTED]' : 'undefined'
} and TLS key: ${autoopsConfig?.api?.tls?.key ? '[REDACTED]' : 'undefined'}
and TLS ca: ${autoopsConfig?.api?.tls?.ca ? '[REDACTED]' : 'undefined'}`
} and TLS key: ${autoopsConfig?.api?.tls?.key ? '[REDACTED]' : 'undefined'}`
);
const tlsConfig = this.createTlsConfig(autoopsConfig);

Expand Down Expand Up @@ -146,7 +145,6 @@ export class AutoOpsAPIService {
enabled: true,
certificate: autoopsConfig?.api?.tls?.certificate,
key: autoopsConfig?.api?.tls?.key,
certificateAuthorities: autoopsConfig?.api?.tls?.ca,
})
);
}
Expand All @@ -164,7 +162,6 @@ export class AutoOpsAPIService {
...requestConfig.httpsAgent.options,
cert: requestConfig.httpsAgent.options.cert ? 'REDACTED' : undefined,
key: requestConfig.httpsAgent.options.key ? 'REDACTED' : undefined,
ca: requestConfig.httpsAgent.options.ca ? 'REDACTED' : undefined,
},
},
});
Expand Down
1 change: 0 additions & 1 deletion x-pack/plugins/data_usage/server/types/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@ export interface AutoOpsConfig {
tls?: {
certificate?: string;
key?: string;
ca?: string;
};
};
}
Expand Down

0 comments on commit c6cddb7

Please sign in to comment.