New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
c_rehash: Fix file extension matching #19427
Conversation
For some reason, parenthesis were added 8 years ago in commit a787c25. This essentially removed the \. and $ constructs from the middle branches. Hence a file called e.g. cert.key would accidentally match the (cer) rule. CLA: trivial
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK with CLA: trivial
okay with trivial |
24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set. Please review the updates and set the label manually. |
Merged to master branch. In the end I've decided to not merge to 3.0 branch as in theory someone could depend on the existing match there and we do not want to introduce a break of such kind in existing stable release. Thank you for your contribution. |
For some reason, parenthesis were added 8 years ago in commit a787c25. This essentially removed the \. and $ constructs from the middle branches. Hence a file called e.g. cert.key would accidentally match the (cer) rule. CLA: trivial Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #19427)
appreciated, thank you! |
For some reason, parenthesis were added 8 years ago in commit a787c25. This essentially removed the \. and $ constructs from the middle branches. Hence a file called e.g. cert.key would accidentally match the (cer) rule. CLA: trivial Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from openssl#19427) (cherry picked from commit 706fc5f)
For some reason, parenthesis were added 8 years ago in commit a787c25. This essentially removed the \. and $ constructs from the middle branches. Hence a file called e.g. cert.key would accidentally match the (cer) rule. CLA: trivial Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from #19427) (cherry picked from commit 706fc5f)
For some reason, parenthesis were added 8 years ago in commit a787c25. This essentially removed the \. and $ constructs from the middle branches. Hence a file called e.g. cert.key would accidentally match the (cer) rule. CLA: trivial Reviewed-by: Paul Dale <pauli@openssl.org> Reviewed-by: Tomas Mraz <tomas@openssl.org> (Merged from openssl#19427)
For some reason, parenthesis were added 8 years ago in commit a787c25. This essentially removed the . and $ constructs from the middle branches. Hence a file called e.g. cert.key would accidentally match the (cer) rule.
CLA: trivial