chocolatey / package-validator

Windows service to validate packages conform to package standards
Apache License 2.0
31 stars 29 forks source link

Erroneous detection of "installArguments" #181

Open bc3tech opened 6 years ago

bc3tech commented 6 years ago

This line causes erroneous detection and failure of packages that use any variable name that contains installArguments as a substring.

Real-world effect can be seen here where the uninstall script uses a registry key called UninstallArguments and is being caught & falsely failed by this validation check.

seems like a better check would be .Contains("$installArguments"), etc

bc3tech commented 6 years ago

Now impacting a stable push here