Skip to content

Commit a0a32ee

Browse files
committed
fix: undefined case
1 parent 50592d7 commit a0a32ee

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

spec/Utils.spec.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -182,6 +182,11 @@ describe('Utils', () => {
182182
expect(error.message).toBe('Permission denied');
183183
});
184184

185+
it('should not crash with config undefined', () => {
186+
const error = createSanitizedError(Parse.Error.OPERATION_FORBIDDEN, 'Detailed error message', undefined);
187+
expect(error.message).toBe('Permission denied');
188+
});
189+
185190
it('should return the detailed message when disableSanitizeError is true', () => {
186191
const config = { disableSanitizeError: true };
187192
const error = createSanitizedError(Parse.Error.OPERATION_FORBIDDEN, 'Detailed error message', config);
@@ -196,6 +201,11 @@ describe('Utils', () => {
196201
expect(error.message).toBe('Permission denied');
197202
});
198203

204+
it('should not crash with config undefined', () => {
205+
const error = createSanitizedHttpError(403, 'Detailed error message', undefined);
206+
expect(error.message).toBe('Permission denied');
207+
});
208+
199209
it('should return the detailed message when disableSanitizeError is true', () => {
200210
const config = { disableSanitizeError: true };
201211
const error = createSanitizedHttpError(403, 'Detailed error message', config);

src/Error.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ function createSanitizedError(errorCode, detailedMessage, config) {
1616
defaultLogger.error(detailedMessage);
1717
}
1818

19-
return new Parse.Error(errorCode, config.disableSanitizeError ? detailedMessage : 'Permission denied');
19+
return new Parse.Error(errorCode, config?.disableSanitizeError ? detailedMessage : 'Permission denied');
2020
}
2121

2222
/**
@@ -37,7 +37,7 @@ function createSanitizedHttpError(statusCode, detailedMessage, config) {
3737

3838
const error = new Error();
3939
error.status = statusCode;
40-
error.message = config.disableSanitizeError ? detailedMessage : 'Permission denied';
40+
error.message = config?.disableSanitizeError ? detailedMessage : 'Permission denied';
4141
return error;
4242
}
4343

0 commit comments

Comments
 (0)