-
Notifications
You must be signed in to change notification settings - Fork 467
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
Fix SCL block parameter substitution #2901
Conversation
After a quoted character, transitioning into CLS_WITHIN_STRING is not enough, because another quote or "string end" character may follow. Signed-off-by: László Várady <laszlo.varady93@gmail.com>
@jszigetvari Could you try this patch with your original config, please? |
Build SUCCESS |
@kira-syslogng retest this please |
Build SUCCESS |
@MrAnno As far as I could check, my test config, that produced the error eariler now passes the syntax validation. |
@jszigetvari Thanks for the test! |
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.
Nice catch!
@kira-syslogng retest this please |
Build SUCCESS |
After fixing the SCL parameter substitution in syslog-ng#2901, the Telegram destination has stopped working. The type of the use-system-cert-store() option should be yes/no, strings are not accepted. Signed-off-by: László Várady <laszlo.varady@protonmail.com>
After a quoted character, transitioning into
CLS_WITHIN_STRING
is notenough, because another quote or "string end" character may follow.
Here is the state machine of
_track_string_state()
, the red lines were missing.Fixes #2879