Skip to content

Potential fix for code scanning alert no. 77: Incomplete string escaping or encoding#7096

Merged
labkey-jeckels merged 7 commits intodevelopfrom
fb_timelineJSCleanup
Oct 4, 2025
Merged

Potential fix for code scanning alert no. 77: Incomplete string escaping or encoding#7096
labkey-jeckels merged 7 commits intodevelopfrom
fb_timelineJSCleanup

Conversation

@labkey-jeckels
Copy link
Contributor

@labkey-jeckels labkey-jeckels commented Oct 2, 2025

Address https://github.com/LabKey/platform/security/code-scanning/77

Improve JS HTML parsing regexs. Likely abandoned code, which I hope we can completely delete soon. We have no automated tests nor docs on expected usage.

labkey-jeckels and others added 2 commits October 2, 2025 10:42
…ing or encoding

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@labkey-jeckels labkey-jeckels self-assigned this Oct 2, 2025
@labkey-jeckels labkey-jeckels requested a review from a team October 2, 2025 17:44
@labkey-jeckels labkey-jeckels marked this pull request as ready for review October 2, 2025 17:45
Copy link
Contributor

@labkey-susanh labkey-susanh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hoping we can just get rid of this code as you say.

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@labkey-jeckels labkey-jeckels merged commit a5ada93 into develop Oct 4, 2025
6 checks passed
@labkey-jeckels labkey-jeckels deleted the fb_timelineJSCleanup branch October 4, 2025 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants