Skip to content

BUILD-9046 fix the check#101

Merged
tomverin merged 1 commit intomasterfrom
feat/tom/BUILD-9046
Sep 9, 2025
Merged

BUILD-9046 fix the check#101
tomverin merged 1 commit intomasterfrom
feat/tom/BUILD-9046

Conversation

@tomverin
Copy link
Copy Markdown
Contributor

@tomverin tomverin commented Sep 9, 2025

No description provided.

@tomverin tomverin requested review from a team and Copilot September 9, 2025 10:05
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds input validation for the optional_checks parameter in the releasability status GitHub Action to prevent injection attacks and ensure data integrity.

  • Added regex validation to restrict optional_checks input to alphanumeric characters, commas, and spaces only
  • Included error handling that exits with failure status when invalid input is detected

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread releasability-status/action.yml
Comment thread releasability-status/action.yml
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Sep 9, 2025

🤖 Pull Request summary

Adds input validation for optional_checks parameter in releasability-status action.

• Validates optional_checks input format using regex pattern for alphanumeric chars, commas, and spaces
• Outputs GitHub Actions error message and exits with code 1 on validation failure

Review focus: Ensure regex pattern ^[a-zA-Z0-9,\ ]*$ matches expected input format requirements.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@tomverin tomverin merged commit 384da7f into master Sep 9, 2025
7 checks passed
@tomverin tomverin deleted the feat/tom/BUILD-9046 branch September 9, 2025 12:01
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.

3 participants