Use replaceAll when replacing separators

[karthiknadig]

See here:
https://github.com/microsoft/vscode-python/security/code-scanning/10?query=ref%3Arefs%2Fheads%2Fmain

Issue Location:
https://github.com/microsoft/vscode-python/blob/2021.1.502429796/src/client/pythonEnvironments/base/info/interpreter.ts
https://github.com/microsoft/vscode-python/blob/2021.1.502429796/src/client/pythonEnvironments/info/interpreter.ts

[KamalSinghKhanna]

Hi, I would like to work on this issue,can you suggest me ways to get started?

[karthiknadig]

@KamalSinghKhanna go ahead

[vijaya-lakshmi-venkatraman]

Hello,
I would like to pick this issue if still available.

So the required changes are :

1. In /pythonEnvironments/base/info/interpreter.ts (Line 73)

// Concat these together to make a set of quoted strings
    const quoted = argv.reduce((p, c) => (p ? `${p} "${c}"` : `"${c.replaceAll('\\', '\\\\')}"`), '');

2. In /pythonEnvironments/info/interpreter.ts (Line 60)

// Concat these together to make a set of quoted strings
    const quoted = argv.reduce((p, c) => (p ? `${p} "${c}"` : `"${c.replace('\\', '\\\\')}"`), '');

Is that right?

[karthiknadig]

@vijaya-lakshmi-venkatraman That is correct.

Edit: You may have to add the replaceAll API to the string prototype. src\client\common\extensions.ts

[vijaya-lakshmi-venkatraman]

Thanks @karthiknadig.
I would like to take this up if @KamalSinghKhanna is not working on this anymore.

[karthiknadig]

@vijaya-lakshmi-venkatraman go ahead.

[KamalSinghKhanna]

Is this issue still open or @vijaya-lakshmi-venkatraman is still working on this issue

[karthiknadig]

One thing to note. there is no replaceAll API. There are two ways one could do this. Either extend string prototype to add a replaceAll. Or use pattern matching to replace all.

if you decide to extend string, you can do that here src\client\common\extensions.ts . Then all you have to do is include it in that file where you consume replaceAll