Skip to content

Make nokogiri optional#40

Draft
ashie wants to merge 2 commits into
masterfrom
rexml
Draft

Make nokogiri optional#40
ashie wants to merge 2 commits into
masterfrom
rexml

Conversation

@ashie

@ashie ashie commented Jun 18, 2026

Copy link
Copy Markdown
Member

Use REXML as runtime fallback.

We are considering to drop Nokogiri from fluent-package in the next major version.
AFAIK only this plugin mandatory depends on Nokogiri in the package.

Probably this plugin isn't so used recent days since windows_eventlog2 plugin doesn't use XML output by default in current version. We will deprecate XML output, so we don't care about performance degradation by replacing nokogiri with REXML. If a user is concerned about this, we will prompt them to install Nokogiri themselves.

Comment thread fluent-plugin-parser-winevt_xml.gemspec Outdated

spec.add_runtime_dependency "fluentd", [">= 0.14.12", "< 2"]
spec.add_runtime_dependency "nokogiri", ">= 1.12.5"
spec.add_runtime_dependency "rexml"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

TODO: Limit the version

Comment thread lib/fluent/plugin/parser_winevt_sax.rb Outdated
@ashie ashie force-pushed the rexml branch 7 times, most recently from 9c2b84e to 1865d25 Compare June 19, 2026 09:01
ashie added 2 commits June 19, 2026 18:12
Use REXML as runtime fallback.

Signed-off-by: Takuro Ashie <ashie@clear-code.com>
Signed-off-by: Takuro Ashie <ashie@clear-code.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant