Skip to content

Commit

Permalink
[Security Solution][Exceptions] Adds error catch to Exceptions toStri…
Browse files Browse the repository at this point in the history
…ng method (elastic#105928)
  • Loading branch information
dplumlee authored and kibanamachine committed Jul 16, 2021
1 parent b26051d commit f5aaf9d
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,9 @@ import {
getProcessCodeSignature,
retrieveAlertOsTypes,
filterIndexPatterns,
getCodeSignatureValue,
} from './helpers';
import { AlertData } from './types';
import { AlertData, Flattened } from './types';
import {
ListOperatorTypeEnum as OperatorTypeEnum,
EntriesArray,
Expand All @@ -41,6 +42,7 @@ import { getCommentsArrayMock } from '../../../../../lists/common/schemas/types/
import { fields } from '../../../../../../../src/plugins/data/common/index_patterns/fields/fields.mocks';
import { ENTRIES, OLD_DATE_RELATIVE_TO_DATE_NOW } from '../../../../../lists/common/constants.mock';
import { IFieldType, IIndexPattern } from 'src/plugins/data/common';
import { CodeSignature } from '../../../../common/ecs/file';

jest.mock('uuid', () => ({
v4: jest.fn().mockReturnValue('123'),
Expand Down Expand Up @@ -340,6 +342,17 @@ describe('Exception helpers', () => {
});
});

describe('#getCodeSignatureValue', () => {
test('it should return empty string if code_signature nested value are undefined', () => {
// Using the unsafe casting because with our types this shouldn't be possible but there have been issues with old data having undefined values in these fields
const payload = ([{ trusted: undefined, subject_name: undefined }] as unknown) as Flattened<
CodeSignature[]
>;
const result = getCodeSignatureValue(payload);
expect(result).toEqual([{ trusted: '', subjectName: '' }]);
});
});

describe('#entryHasNonEcsType', () => {
const mockEcsIndexPattern = {
title: 'testIndex',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -325,8 +325,8 @@ export const getCodeSignatureValue = (
if (Array.isArray(codeSignature) && codeSignature.length > 0) {
return codeSignature.map((signature) => {
return {
subjectName: signature.subject_name ?? '',
trusted: signature.trusted.toString() ?? '',
subjectName: signature?.subject_name ?? '',
trusted: signature?.trusted?.toString() ?? '',
};
});
} else {
Expand Down

0 comments on commit f5aaf9d

Please sign in to comment.