phlux/mattermostest
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[MM-54465] Remove lowercase call when checking path names (#2831)

Browse Source
This commit is contained in:
Devin Binnie
2023-09-15 10:20:28 -04:00
committed by GitHub
parent ce1447591d
commit 128d15a283
4 changed files with 4 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/common/Validator.ts
View File

@@ -201,7 +201,7 @@ export function validateV0ConfigData(data: ConfigV0) {
function cleanURL(url: string): string {
let updatedURL = url;
if (updatedURL.includes('\\')) {
updatedURL = updatedURL.toLowerCase().replace(/\\/gi, '/');
updatedURL = updatedURL.replace(/\\/gi, '/');
}
return updatedURL;
}

7
src/common/utils/url.test.js
View File

@@ -25,14 +25,9 @@ jest.mock('common/views/View', () => ({
describe('common/utils/url', () => {
describe('getFormattedPathName', () => {
it('should format all to lower case', () => {
const unformattedPathName = '/aAbBbB/cC/DdeR/';
expect(getFormattedPathName(unformattedPathName)).toBe('/aabbbb/cc/dder/');
});
it('should add trailing slash', () => {
const unformattedPathName = '/aAbBbB/cC/DdeR';
expect(getFormattedPathName(unformattedPathName)).toBe('/aabbbb/cc/dder/');
expect(getFormattedPathName(unformattedPathName)).toBe('/aAbBbB/cC/DdeR/');
});
});
describe('parseURL', () => {

2
src/common/utils/url.ts
View File

@@ -6,7 +6,7 @@ import {isHttpsUri, isHttpUri, isUri} from 'valid-url';
import buildConfig from 'common/config/buildConfig';
import {customLoginRegexPaths, nonTeamUrlPaths, CALLS_PLUGIN_ID} from 'common/utils/constants';
export const getFormattedPathName = (pn: string) => (pn.endsWith('/') ? pn.toLowerCase() : `${pn.toLowerCase()}/`);
export const getFormattedPathName = (pn: string) => (pn.endsWith('/') ? pn : `${pn}/`);
export const parseURL = (inputURL: string | URL) => {
if (inputURL instanceof URL) {
return inputURL;

2
src/main/windows/callsWidgetWindow.test.js
View File

@@ -241,7 +241,7 @@ describe('main/windows/callsWidgetWindow', () => {
beforeEach(() => {
urlUtils.parseURL.mockImplementation((url) => new URL(url));
urlUtils.getFormattedPathName.mockImplementation((pn) => {
return pn.endsWith('/') ? pn.toLowerCase() : `${pn.toLowerCase()}/`;
return pn.endsWith('/') ? pn : `${pn}/`;
});
callsWidgetWindow.options = {
callID: 'test-call-id',