mrva/vscode-codeql
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Avoid using path.join for sarif uris

Browse Source 
These are uris, not paths and always use '/', even on windows.
This commit is contained in:
Andrew Eisenberg
2020-09-29 14:58:16 -07:00
parent 03f1e4ef08
commit c34c9fae6a
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
extensions/ql-vscode/src/sarif-utils.ts
View File

@@ -1,5 +1,4 @@
import * as Sarif from 'sarif'; import * as Sarif from 'sarif';
import * as path from 'path';
import { ResolvableLocationValue } from './bqrs-cli-types'; import { ResolvableLocationValue } from './bqrs-cli-types';
export interface SarifLink { export interface SarifLink {
@@ -74,9 +73,7 @@ export function getPathRelativeToSourceLocationPrefix(
sarifRelativeUri: string sarifRelativeUri: string
) { ) {
const normalizedSourceLocationPrefix = sourceLocationPrefix.replace(/\\/g, '/'); const normalizedSourceLocationPrefix = sourceLocationPrefix.replace(/\\/g, '/');
return `file:${ return `file:${normalizedSourceLocationPrefix}/${sarifRelativeUri}`;
path.join(normalizedSourceLocationPrefix, sarifRelativeUri)
}`;
} }
export function parseSarifLocation( export function parseSarifLocation(