Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -76,12 +76,32 @@ const AuthenticationMethods: React.FC<{ method: string }> = ({ method }) => {
);
case 'SASL/AWS IAM':
return (
<Input
label="AWS Profile Name"
type="text"
name="auth.props.awsProfileName"
withError
/>
<>
<Input
label="AWS Profile Name"
type="text"
name="auth.props.awsProfileName"
withError
/>
<Input
label="AWS Role Arn"
type="text"
name="auth.props.awsRoleArn"
withError
/>
<Input
label="AWS Role Session Name"
type="text"
name="auth.props.awsRoleSessionName"
withError
/>
<Input
label="AWS STS Region"
type="text"
name="auth.props.awsStsRegion"
withError
/>
</>
);
case 'mTLS':
return <SSLForm prefix="auth.keystore" title="Keystore" />;
Expand Down
5 changes: 4 additions & 1 deletion frontend/src/widgets/ClusterConfigForm/schema.ts
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,10 @@ const authPropsSchema = lazy((_, { parent }) => {
});
case 'SASL/AWS IAM':
return object({
awsProfileName: string(),
awsProfileName: string().notRequired(),
awsRoleArn: string().notRequired(),
awsRoleSessionName: string().notRequired(),
awsStsRegion: string().notRequired(),
});
case 'SASL/Azure Entra':
case 'SASL/GCP IAM':
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -260,6 +260,9 @@
'software.amazon.msk.auth.iam.IAMClientCallbackHandler',
'sasl.jaas.config': getJaasConfig('SASL/AWS IAM', {
awsProfileName: props.awsProfileName,
awsRoleArn: props.awsRoleArn,
awsRoleSessionName: props.awsRoleSessionName,
awsStsRegion: props.awsStsRegion

Check warning on line 265 in frontend/src/widgets/ClusterConfigForm/utils/transformFormDataToPayload.ts

View workflow job for this annotation

GitHub Actions / build-and-test / tests

Insert `,`
}),
};
break;
Expand Down
Loading