Skip to content
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

Invalid handling of dot (.) and asterisk (*) in config file name and path for include clause [CORE5237] #5516

Closed
firebird-issue-importer opened this issue May 11, 2016 · 8 comments

Comments

@firebird-issue-importer

Submitted by: @aafemt

Assigned to: @aafemt

These statements don't work as expected:
include *\*.conf
include .*\*.conf
include .\*\*.conf
include .\*.conf
include .\.conf\*.conf

Following one works:
include .conf\*.conf

Commits: 739b993 fc1ebda

====== Test Details ======

fbtest does not allow to restart FB instance.

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 11, 2016

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 11, 2016

Modified by: Sean Leyne (seanleyne)

summary: Invalid handling of dot and asteric in config file name and path for include clause => Invalid handling of dot (.) and asterisk (*) in config file name and path for include clause

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 12, 2016

Modified by: @aafemt

assignee: Alexander Peshkov [ alexpeshkoff ] => Dimitry Sibiryakov [ aafemt ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented May 27, 2016

Modified by: @aafemt

status: Open [ 1 ] => In Progress [ 3 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 11, 2016

Modified by: @aafemt

status: In Progress [ 3 ] => Open [ 1 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 11, 2016

Modified by: @aafemt

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 4.0 Alpha 1 [ 10731 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 16, 2016

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Resolved [ 5 ]

QA Status: No test => Cannot be tested

Test Details: fbtest does not allow to restart FB instance.

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jun 16, 2016

Modified by: @pavel-zotov

status: Resolved [ 5 ] => Closed [ 6 ]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant