fix: subs all certs fields required (#5891)

* fix: all three certs fields are requried

* fix: lint

Author: helenosheaa

src/writeData/subscriptions/components/CertificateInput.tsx

@@ -83,6 +83,7 @@ const NewCertificateInput: FC<OwnProps> = ({updateForm, subscription}) => {
         placeholder={CertificatePlaceholders.clientKey}
         value={subscription?.brokerClientKey ?? ''}
         rows={4}
+        required
       />
       <TextAreaWithLabel
         name="CertificateAuthority"
@@ -91,6 +92,7 @@ const NewCertificateInput: FC<OwnProps> = ({updateForm, subscription}) => {
         value={subscription?.brokerClientCert ?? ''}
         placeholder={CertificatePlaceholders.clientCert}
         rows={4}
+        required
       />
     </FlexBox>
   )

src/writeData/subscriptions/utils/form.ts

@@ -248,9 +248,8 @@ const checkBasicSelected = (form: Subscription): boolean =>
 const checkCertificateRequiredFields = (form: Subscription): boolean =>
   form.authType === BrokerAuthTypes.Certificate &&
   !!form.brokerCACert &&
-  // you either need to provide both or neither
-  ((!!form.brokerClientCert && !!form.brokerClientKey) ||
-    (!form.brokerClientCert && !form.brokerClientKey))
+  !!form.brokerClientCert &&
+  !!form.brokerClientKey
 
 const checkCreatingCertificate = (form: Subscription): boolean =>
   form.authType === BrokerAuthTypes.Certificate && !form.brokerCertCreationDate