convert 'None' and '' string values to NoneType before forwarding the…

[T0MASD]

We had some issues importing logs that contained "None" string values, converting them NoneTypes did the trick.
[2017-05-22 03:41:43,092][DEBUG ][action.bulk ] [logstash-2017.05.22][2] failed to execute bulk item (index) index { [logstash-2017.05.22][fluentd][AVwuPw0zO--t2Sy8RmXq],source[{ "Status": "None", ...
Caused by: org.elasticsearch.index.mapper.MapperParsingException: object mapping for [fluentd] tried to parse field [status] as object, but got EOF, has a concrete value been provided to it?
I'm not sure where it's best to place the nonevalues=['None',''] to be used in _add_dic

[coveralls]

Coverage increased (+0.04%) to 90.865% when pulling b290dbd on T0MASD:convert_none_strings into 1c69535 on fluent:master.

[coveralls]

Coverage increased (+0.04%) to 90.865% when pulling b290dbd on T0MASD:convert_none_strings into 1c69535 on fluent:master.

[coveralls]

Coverage increased (+0.04%) to 90.865% when pulling 7b1a26d on T0MASD:convert_none_strings into 1c69535 on fluent:master.

[coveralls]

Coverage increased (+0.04%) to 90.865% when pulling 7b1a26d on T0MASD:convert_none_strings into 1c69535 on fluent:master.

[coveralls]

Coverage increased (+0.04%) to 90.865% when pulling 7b1a26d on T0MASD:convert_none_strings into 1c69535 on fluent:master.

[coveralls]

Coverage increased (+0.04%) to 90.865% when pulling 7b1a26d on T0MASD:convert_none_strings into 1c69535 on fluent:master.

[repeatedly]

Sorry for the late reply.
How about setting this values via class variable like below?

fluent.handler.FluentHandler.NoneValues = ["None"] # default is None

Changing default behaviour may break existing environment.