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

Configuration fbtrace.conf contains syntax errors [CORE4163] #4490

Closed
firebird-issue-importer opened this issue Aug 4, 2013 · 6 comments
Closed

Comments

@firebird-issue-importer
Copy link

firebird-issue-importer commented Aug 4, 2013

Submitted by: @mrotteveel

Relate to CORE4219

Firebird.log contains errors when I change firebird conf to enable system auditing with AuditTraceConfigFile = fbtrace.conf

RAMONA Sun Aug 04 15:13:20 2013
Trace plugin fbtrace returned error on call trace_create.
error while parsing trace configuration
line 208: error while compiling regular expression "%[\/](test|azk2|rulez).fdb"

The entry in the file is the one that is in the firebird 3 distribution:

#⁠ Enable logging for test.fdb, azk2.fdb and rulez.fdb in any directory
#⁠ into log file name matching database name - test.log, azk2.log and
#⁠ rulez.log appropriately
#⁠
database = %[\\/](test|azk2|rulez).fdb
{
enabled = true
log_filename = \1.log
}

Either the syntax used in the config file is wrong or the parser doesn't process it correctly.

Commits: b84835d FirebirdSQL/fbt-repository@927084b

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Sep 17, 2013

Modified by: @AlexPeshkoff

assignee: Alexander Peshkov [ alexpeshkoff ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Sep 17, 2013

Modified by: @AlexPeshkoff

Link: This issue relate to CORE4219 [ CORE4219 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Sep 18, 2013

Commented by: @AlexPeshkoff

One error was due to the reason, fixed in related CORE4219.
Next is fixed here - they happened because FB3 config sorted config keys in ABC order, which is wrong for trace config files.

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Sep 18, 2013

Modified by: @AlexPeshkoff

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

resolution: Fixed [ 1 ]

Fix Version: 3.0 Alpha 2 [ 10560 ]

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Mar 27, 2014

Modified by: @pcisar

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

@firebird-issue-importer
Copy link
Author

firebird-issue-importer commented Jan 18, 2016

Modified by: @pavel-zotov

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

QA Status: No test

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

No branches or pull requests

2 participants