Fix dashboard stats and replace basic constraints with subject type p…

…roperty
CZERTAINLY · Nov 7, 2024 · 5e9dd8f · 5e9dd8f
1 parent b614e38
commit 5e9dd8f
Show file tree

Hide file tree

Showing 11 changed files with 103 additions and 21 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/components/_pages/certificates/CertificateStatus/index.tsx b/src/components/_pages/certificates/CertificateStatus/index.tsx
@@ -2,6 +2,7 @@ import { Badge } from 'reactstrap';
 import {
     CertificateEventHistoryDtoStatusEnum,
     CertificateState,
+    CertificateSubjectType,
     CertificateValidationStatus,
     ComplianceRuleStatus,
     ComplianceStatus,
@@ -10,7 +11,13 @@ import { getCertificateStatusColor, useGetStatusText } from 'utils/certificate';
 import { capitalize } from 'utils/common-utils';
 
 interface Props {
-    status: CertificateState | CertificateValidationStatus | CertificateEventHistoryDtoStatusEnum | ComplianceStatus | ComplianceRuleStatus;
+    status:
+        | CertificateState
+        | CertificateValidationStatus
+        | CertificateEventHistoryDtoStatusEnum
+        | ComplianceStatus
+        | ComplianceRuleStatus
+        | CertificateSubjectType;
     asIcon?: boolean;
 }
 

diff --git a/src/components/_pages/certificates/detail/index.tsx b/src/components/_pages/certificates/detail/index.tsx
@@ -26,6 +26,7 @@ import {
     CertificateFormatEncoding,
     CertificateRequestFormat,
     CertificateRevocationReason,
+    CertificateSubjectType,
     CertificateValidationStatus,
 } from '../../../../types/openapi';
 
@@ -1422,8 +1423,8 @@ export default function CertificateDetail() {
                       ],
                   },
                   {
-                      id: 'basicConstraint',
-                      columns: ['Basic Constraint', certificate.basicConstraints],
+                      id: 'subjectType',
+                      columns: ['Subject Type', <CertificateStatus status={certificate.subjectType} />],
                   },
               ];
         if (certificate?.state !== CertStatus.Requested) {
@@ -1437,7 +1438,7 @@ export default function CertificateDetail() {
             certDetail.unshift({
                 id: 'trustedCa',
                 columns: [
-                    certificate?.basicConstraints?.includes('End Entity') ? 'Trusted Self-Signed' : 'Trusted CA',
+                    certificate?.subjectType == CertificateSubjectType.SelfSignedEndEntity ? 'Trusted Self-Signed' : 'Trusted CA',
                     <SwitchWidget disabled={isUpdatingTrustedStatus} checked={certificate.trustedCa ?? false} onClick={switchCallback} />,
                 ],
             });

diff --git a/src/components/_pages/dashboard/index.tsx b/src/components/_pages/dashboard/index.tsx
@@ -293,10 +293,26 @@ function Dashboard() {
 
                 <Col>
                     <DonutChart
-                        title={'Certificates by Constraints'}
-                        data={dashboard?.certificateStatByBasicConstraints}
+                        title={'Certificates by Subject type'}
+                        data={dashboard?.certificateStatBySubjectType}
                         entity={EntityType.CERTIFICATE}
-                        onSetFilter={(_index, _labels) => []}
+                        onSetFilter={(index, labels) => {
+                            const certificateSubjectTypeEnum = platformEnums?.CertificateSubjectType;
+                            const certificateSubjectTypeList = Object.keys(certificateSubjectTypeEnum).map(
+                                (key) => certificateSubjectTypeEnum[key],
+                            );
+                            const selectedCertificateSubjectType = certificateSubjectTypeList.find(
+                                (status) => status.label === labels[index],
+                            );
+                            return [
+                                {
+                                    fieldSource: FilterFieldSource.Property,
+                                    condition: FilterConditionOperator.Equals,
+                                    fieldIdentifier: 'SUBJECT_TYPE',
+                                    value: selectedCertificateSubjectType?.code ? [selectedCertificateSubjectType?.code] : [''],
+                                },
+                            ];
+                        }}
                         redirect="../certificates"
                     />
                 </Col>

diff --git a/src/types/openapi/.openapi-generator/FILES b/src/types/openapi/.openapi-generator/FILES
@@ -120,6 +120,7 @@ models/CertificateRequestFormat.ts
 models/CertificateResponseDto.ts
 models/CertificateRevocationReason.ts
 models/CertificateState.ts
+models/CertificateSubjectType.ts
 models/CertificateType.ts
 models/CertificateUpdateObjectsDto.ts
 models/CertificateValidationCheck.ts

diff --git a/src/types/openapi/models/CertificateDetailDto.ts b/src/types/openapi/models/CertificateDetailDto.ts
@@ -3,7 +3,7 @@
  * CZERTAINLY Core API
  * REST API for CZERTAINLY Core
  *
- * The version of the OpenAPI document: 2.12.1-SNAPSHOT
+ * The version of the OpenAPI document: 2.13.1-SNAPSHOT
  * Contact: [email protected]
  *
  * NOTE: This class is auto generated by OpenAPI Generator (https://openapi-generator.tech).
@@ -17,6 +17,7 @@ import type {
     CertificateProtocolDto,
     CertificateRequestDto,
     CertificateState,
+    CertificateSubjectType,
     CertificateType,
     CertificateValidationStatus,
     ComplianceStatus,
@@ -185,11 +186,10 @@ export interface CertificateDetailDto {
      */
     keyUsage: Array<string>;
     /**
-     * Basic Constraints
-     * @type {string}
+     * @type {CertificateSubjectType}
      * @memberof CertificateDetailDto
      */
-    basicConstraints: string;
+    subjectType: CertificateSubjectType;
     /**
      * Certificate metadata
      * @type {Array<MetadataResponseDto>}

diff --git a/src/types/openapi/models/CertificateSubjectType.ts b/src/types/openapi/models/CertificateSubjectType.ts
@@ -0,0 +1,25 @@
+// tslint:disable
+/**
+ * CZERTAINLY Core API
+ * REST API for CZERTAINLY Core
+ *
+ * The version of the OpenAPI document: 2.13.1-SNAPSHOT
+ * Contact: [email protected]
+ *
+ * NOTE: This class is auto generated by OpenAPI Generator (https://openapi-generator.tech).
+ * https://openapi-generator.tech
+ * Do not edit the class manually.
+ */
+
+/**
+ * Certificate subject type
+ * @export
+ * @enum {string}
+ */
+export enum CertificateSubjectType {
+    EndEntity = 'endEntity',
+    SelfSignedEndEntity = 'selfSignedEndEntity',
+    IntermediateCa = 'intermediateCa',
+    RootCa = 'rootCa'
+}
+
diff --git a/src/types/openapi/models/PlatformEnum.ts b/src/types/openapi/models/PlatformEnum.ts
@@ -3,7 +3,7 @@
  * CZERTAINLY Core API
  * REST API for CZERTAINLY Core
  *
- * The version of the OpenAPI document: 2.12.1-SNAPSHOT
+ * The version of the OpenAPI document: 2.13.1-SNAPSHOT
  * Contact: [email protected]
  *
  * NOTE: This class is auto generated by OpenAPI Generator (https://openapi-generator.tech).
@@ -37,6 +37,7 @@ export enum PlatformEnum {
     CertificateRequestFormat = 'CertificateRequestFormat',
     DiscoveryStatus = 'DiscoveryStatus',
     CertificateProtocol = 'CertificateProtocol',
+    CertificateSubjectType = 'CertificateSubjectType',
     KeyAlgorithm = 'KeyAlgorithm',
     KeyFormat = 'KeyFormat',
     KeyState = 'KeyState',

diff --git a/src/types/openapi/models/StatisticsDto.ts b/src/types/openapi/models/StatisticsDto.ts
@@ -3,7 +3,7 @@
  * CZERTAINLY Core API
  * REST API for CZERTAINLY Core
  *
- * The version of the OpenAPI document: 2.12.1-SNAPSHOT
+ * The version of the OpenAPI document: 2.13.1-SNAPSHOT
  * Contact: [email protected]
  *
  * NOTE: This class is auto generated by OpenAPI Generator (https://openapi-generator.tech).
@@ -101,11 +101,11 @@ export interface StatisticsDto {
      */
     certificateStatByKeySize?: { [key: string]: number; };
     /**
-     * Map of Certificate count by basic constraints
+     * Map of Certificate count by subject type
      * @type {{ [key: string]: number; }}
      * @memberof StatisticsDto
      */
-    certificateStatByBasicConstraints?: { [key: string]: number; };
+    certificateStatBySubjectType?: { [key: string]: number; };
     /**
      * Map of Certificate count by state
      * @type {{ [key: string]: number; }}

diff --git a/src/types/openapi/models/index.ts b/src/types/openapi/models/index.ts
@@ -76,6 +76,7 @@ export * from './CertificateRequestFormat';
 export * from './CertificateResponseDto';
 export * from './CertificateRevocationReason';
 export * from './CertificateState';
+export * from './CertificateSubjectType';
 export * from './CertificateType';
 export * from './CertificateUpdateObjectsDto';
 export * from './CertificateValidationCheck';

diff --git a/src/utils/certificate.tsx b/src/utils/certificate.tsx
@@ -5,6 +5,7 @@ import { CertificateDetailResponseModel } from 'types/certificate';
 import {
     CertificateEventHistoryDtoStatusEnum,
     CertificateState,
+    CertificateSubjectType,
     CertificateType,
     CertificateValidationStatus,
     ComplianceRuleStatus,
@@ -27,7 +28,7 @@ export const emptyCertificate: CertificateDetailResponseModel = {
     keySize: -1,
     keyUsage: [],
     extendedKeyUsage: [],
-    basicConstraints: '',
+    subjectType: CertificateSubjectType.EndEntity,
     state: CertificateState.PendingIssue,
     validationStatus: CertificateValidationStatus.NotChecked,
     fingerprint: '',
@@ -70,7 +71,13 @@ export function downloadFile(content: any, fileName: string) {
 }
 
 export function getCertificateStatusColor(
-    status: CertificateState | CertificateValidationStatus | CertificateEventHistoryDtoStatusEnum | ComplianceStatus | ComplianceRuleStatus,
+    status:
+        | CertificateState
+        | CertificateValidationStatus
+        | CertificateEventHistoryDtoStatusEnum
+        | ComplianceStatus
+        | ComplianceRuleStatus
+        | CertificateSubjectType,
 ) {
     switch (status) {
         case CertificateState.Requested:
@@ -130,6 +137,15 @@ export function getCertificateStatusColor(
         case CertificateEventHistoryDtoStatusEnum.Success:
             return '#1ab394';
 
+        case CertificateSubjectType.EndEntity:
+            return '#6c757d';
+        case CertificateSubjectType.SelfSignedEndEntity:
+            return '#f37d63';
+        case CertificateSubjectType.IntermediateCa:
+            return '#3754a5';
+        case CertificateSubjectType.RootCa:
+            return '#1ab394';
+
         default:
             return '#6c757d';
     }
@@ -140,6 +156,7 @@ export function useGetStatusText() {
     const certificateValidationStatusEnum = useSelector(enumSelectors.platformEnum(PlatformEnum.CertificateValidationStatus));
     const complianceStatusEnum = useSelector(enumSelectors.platformEnum(PlatformEnum.ComplianceStatus));
     const complianceRuleStatusEnum = useSelector(enumSelectors.platformEnum(PlatformEnum.ComplianceRuleStatus));
+    const certificateSubjectTypeEnum = useSelector(enumSelectors.platformEnum(PlatformEnum.CertificateSubjectType));
 
     return useCallback(
         (
@@ -148,7 +165,8 @@ export function useGetStatusText() {
                 | CertificateValidationStatus
                 | CertificateEventHistoryDtoStatusEnum
                 | ComplianceStatus
-                | ComplianceRuleStatus,
+                | ComplianceRuleStatus
+                | CertificateSubjectType,
         ) => {
             switch (status) {
                 case CertificateValidationStatus.Valid:
@@ -187,11 +205,23 @@ export function useGetStatusText() {
                 case ComplianceRuleStatus.Na:
                     return getEnumLabel(complianceRuleStatusEnum, status);
 
+                case CertificateSubjectType.EndEntity:
+                case CertificateSubjectType.SelfSignedEndEntity:
+                case CertificateSubjectType.IntermediateCa:
+                case CertificateSubjectType.RootCa:
+                    return getEnumLabel(certificateSubjectTypeEnum, status);
+
                 default:
                     return 'Unknown';
             }
         },
-        [certificateStatusEnum, certificateValidationStatusEnum, complianceStatusEnum, complianceRuleStatusEnum],
+        [
+            certificateStatusEnum,
+            certificateValidationStatusEnum,
+            complianceStatusEnum,
+            complianceRuleStatusEnum,
+            certificateSubjectTypeEnum,
+        ],
     );
 }