merge visit retrives #478
merge visit retrives #478
Conversation
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| }) | ||
| ) | ||
| : this.httpClient.get( | ||
| `icare/visit?${parametersString.replace( |
Check failure
Code scanning / CodeQL
Incomplete string escaping or encoding High
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix AI 3 months ago
To fix the problem, we need to ensure that all occurrences of the & character in parametersString are replaced. The best way to achieve this is by using a regular expression with the global flag (g). This will ensure that every instance of the & character is replaced, not just the first one.
We will modify the replace method call on line 370 to use a regular expression with the global flag. This change will ensure that all occurrences of the & character are replaced with an empty string.
| @@ -369,6 +369,6 @@ | ||
| : this.httpClient.get( | ||
| `icare/visit?${parametersString.replace( | ||
| "&", | ||
| "" | ||
| )}&startIndex=${startIndex}&limit=${limit}` | ||
| `icare/visit?${parametersString.replace( | ||
| /&/g, | ||
| "" | ||
| )}&startIndex=${startIndex}&limit=${limit}` | ||
| ) |
No description provided.