Skip to content

Potential fix for code scanning alert no. 30: Incomplete string escaping or encoding#5677

Merged
WANZARGEN merged 1 commit intodevelopfrom
feature-security
Mar 13, 2025
Merged

Potential fix for code scanning alert no. 30: Incomplete string escaping or encoding#5677
WANZARGEN merged 1 commit intodevelopfrom
feature-security

Conversation

@WANZARGEN
Copy link
Member

Potential fix for https://github.com/cloudforet-io/console/security/code-scanning/30

To fix the problem, we need to ensure that all leading newline characters are removed from the string result. The best way to achieve this is by using a regular expression with the global flag (g) to match all occurrences of the newline character at the beginning of the string. This will ensure that all leading newline characters are removed, regardless of how many there are.

Suggested fixes powered by Copilot Autofix. Review carefully before merging.

…ing or encoding

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Signed-off-by: WANJIN <wanzargen@gmail.com>
@vercel
Copy link

vercel bot commented Mar 6, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
inventory-v2 🔄 Building (Inspect) Visit Preview 💬 Add feedback Mar 6, 2025 4:01am
3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
console ⬜️ Ignored (Inspect) Visit Preview Mar 6, 2025 4:01am
dashboard ⬜️ Ignored (Inspect) Visit Preview Mar 6, 2025 4:01am
web-storybook ⬜️ Ignored (Inspect) Visit Preview Mar 6, 2025 4:01am

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2025

🎉 @piggggggggy and @seungyeoneeee have been randomly selected as the reviewers! Please review. 🙏

@WANZARGEN WANZARGEN marked this pull request as ready for review March 6, 2025 04:22
Copy link
Contributor

@seungyeoneeee seungyeoneeee left a comment

Choose a reason for hiding this comment

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

LGTM

@WANZARGEN WANZARGEN merged commit b25b2ff into develop Mar 13, 2025
11 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants