Skip to content

Conversation

@yugeeklab
Copy link

@yugeeklab yugeeklab commented Apr 30, 2025

DESCRIPTION HERE

(check all that apply)

  • tests added
  • tests passing
  • README updated (if needed)
  • README Table of Contents updated (if needed)
  • History.md and version in gemspec are untouched
  • backward compatible
  • feature works in elasticsearch_dynamic (not required but recommended)

After upgrading the elasticsearch Ruby gem to v9.0.1, Fluentd’s fluent-plugin-elasticsearch started raising:

Invalid media-type value on headers [Content-Type, Accept]

See: uken/fluent-plugin-elasticsearch#1061
elastic/elasticsearch-ruby#2660

To restore compatibility:

For users on Elasticsearch-gem 8.x, continue shipping plugin v5.x

For users on Elasticsearch-gem 9.x, bump plugin to v6.x

This change pins the appropriate plugin version to each major gem series, preventing the header‐validation error.

@yugeeklab
Copy link
Author

@cosmo0920

Copy link
Collaborator

@cosmo0920 cosmo0920 left a comment

Choose a reason for hiding this comment

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

We don't want to lock versions for dependent gems. This could be achieved by Gemfile.
https://docs.fluentd.org/deployment/plugin-management#gemfile-option

@cosmo0920 cosmo0920 closed this May 1, 2025
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.

3 participants