Skip to content

BUILD-9046: fix input validation#102

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

BUILD-9046: fix input validation#102
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.

Copilot AI review requested due to automatic review settings September 9, 2025 12:16
@tomverin tomverin requested a review from a team as a code owner September 9, 2025 12:16
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 updates input validation for version format checking to support an optional build number component in addition to the standard semantic version format.

  • Updated regex pattern to accept both X.Y.Z and X.Y.Z.BUILD version formats
  • Modified error message to reflect the expanded accepted version formats

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

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Sep 9, 2025

🤖 Pull Request summary

Updates version validation regex to support optional 4th build number component.

• Modified regex pattern to accept both X.Y.Z and X.Y.Z.BUILD formats
• Updated error message to reflect new supported version formats with examples

Review focus: Verify the regex correctly handles edge cases and maintains security against injection attacks.

💬 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 enabled auto-merge (squash) September 9, 2025 12:32
@tomverin tomverin merged commit 83fd406 into master Sep 9, 2025
7 checks passed
@tomverin tomverin deleted the feat/tom/BUILD-9046 branch September 9, 2025 12:37
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