Skip to content

rsyslog - skip empty field support#1651

Merged
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
nhosoi:rsyslog_patch
May 21, 2019
Merged

rsyslog - skip empty field support#1651
openshift-merge-robot merged 1 commit intoopenshift:masterfrom
nhosoi:rsyslog_patch

Conversation

@nhosoi
Copy link
Contributor

@nhosoi nhosoi commented May 17, 2019

libfastjson - Adding "void fjson_object_array_del_idx(struct
fjson_object *jso, int idx)", which deletes the idx-th element
in the array type object.

upstream PR: rsyslog/libfastjson#155

liblognorm - Adding a parameter skipempty to the json field type.
If skipempty is set as follows, empty json objects are dropped
from the parsed result.
%field_name:json:skipempty%

If any parameter other than "skipempty" is given ("bogus" in this
example), an error message "invalid flag for JSON parser: bogus"
is issued.

Add test for the skipempty parameter.

upstream PR: rsyslog/liblognorm#305

libfastjson - Adding "void fjson_object_array_del_idx(struct
 fjson_object *jso, int idx)", which deletes the idx-th element
 in the array type object.

 upstream PR: rsyslog/libfastjson#155

liblognorm - Adding a parameter skipempty to the json field type.
 If skipempty is set as follows, empty json objects are dropped
 from the parsed result.
  %field_name:json:skipempty%

 If any parameter other than "skipempty" is given ("bogus" in this
 example), an error message "invalid flag for JSON parser: bogus"
 is issued.

 Add test for the skipempty parameter.

 upstream PR: rsyslog/liblognorm#305
@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label May 17, 2019
Copy link
Contributor

@richm richm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label May 17, 2019
@richm
Copy link
Contributor

richm commented May 21, 2019

/retest

@openshift-merge-robot openshift-merge-robot merged commit 36bff74 into openshift:master May 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants