Skip to content

Commit e88bdc5

Browse files
Copilotlaeubi
andcommitted
Remove validation check as suggested in code review
Co-authored-by: laeubi <[email protected]>
1 parent 1198be7 commit e88bdc5

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

.github/workflows/checkVersions.yml

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -94,16 +94,9 @@ jobs:
9494
if [[ $(git diff --name-only --cached) != '' ]]; then
9595
# Relevant files were staged, i.e. some version were changed
9696
97-
# Validate property name to prevent injection
98-
propertyName="${{ inputs.currentStreamVersionProperty }}"
99-
if [[ ! "$propertyName" =~ ^[a-zA-Z0-9._-]+$ ]]; then
100-
echo "::error::Invalid property name: $propertyName. Must contain only alphanumeric characters, dots, hyphens, and underscores."
101-
exit 1
102-
fi
103-
10497
# Read property as stream version
10598
pushd ${{ inputs.working-directory }}
106-
mvn help:evaluate -Dexpression="$propertyName" ${{ inputs.extra-maven-args }} --quiet '-Doutput=currentStreamVersion-value.txt'
99+
mvn help:evaluate -Dexpression=${{ inputs.currentStreamVersionProperty }} ${{ inputs.extra-maven-args }} --quiet '-Doutput=currentStreamVersion-value.txt'
107100
streamVersion=$(<currentStreamVersion-value.txt)
108101
rm -f currentStreamVersion-value.txt
109102
popd

0 commit comments

Comments
 (0)