Skip to content
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

'fiber-storage' outputs warning: Fiber#storage has borked keys and is being monkey-patched. #4

Closed
daipom opened this issue May 13, 2024 · 7 comments
Milestone

Comments

@daipom
Copy link

daipom commented May 13, 2024

Abstraction

From fluent/fluentd#4488.

Looks like fiber-local uses fiber-storage from v1.1.0.
fiber-storage outputs the following warning message when require:

Fiber#storage has borked keys and is being monkey-patched.

fiber-local v1.0 does not output this warning because it does not use fiber-storage.

Fluentd uses console gem, and console v1.25 uses fiber-local v1.1.
So, this warning message occurs every time when executing Fluentd.

Is this a bug of fiber-storage or fiber-local?
Or is there something wrong with the usage?

(I can't find fiber-storage GitHub repository: https://github.com/socketry/fiber-storage, so I have made this issue here.)

How to reproduce

Ruby: ruby 3.2.2 (2023-03-30 revision e51014f9c0) [x86_64-linux]
OS: Ubuntu 20.04

fiber-storage:

$ gem install fiber-storage
$ gem list fiber-storage

*** LOCAL GEMS ***

fiber-storage (0.1.0)
$ ruby -e "require 'fiber/storage'"
Fiber#storage has borked keys and is being monkey-patched.

fiber-local:

$ gem install fiber-local -v 1.1.0
$ gem list fiber-local

*** LOCAL GEMS ***

fiber-local (1.1.0)
$ ruby -e "require 'fiber/local'"
Fiber#storage has borked keys and is being monkey-patched.
$ gem uninstall fiber-local -v 1.1.0
$ gem install fiber-local -v 1.0.0
$ gem list fiber-local

*** LOCAL GEMS ***

fiber-local (1.0.0)
$ ruby -e "require 'fiber/local'"
(no warning)

Fluentd:

$ git clone [email protected]:fluent/fluentd.git
$ cd fluentd
$ bundle
$ bundle exec gem list console

*** LOCAL GEMS ***

console (1.25.2)
$ bundle exec gem list fiber-local

*** LOCAL GEMS ***

fiber-local (1.1.0)
$ bundle exec fluentd --version
Fiber#storage has borked keys and is being monkey-patched.
fluentd 1.17.0
@daipom
Copy link
Author

daipom commented Jun 13, 2024

@ioquatix If you have time, could you please take a look at this?

@ioquatix
Copy link
Member

Sorry, I missed this. I'll take a look. Maybe we don't need to emit such a warning.

@daipom
Copy link
Author

daipom commented Jun 14, 2024

Thanks! Let me know if there is anything I can help!

@ioquatix ioquatix added this to the v0 milestone Jun 15, 2024
@ioquatix
Copy link
Member

Okay, I've removed the warning.

@daipom
Copy link
Author

daipom commented Jun 17, 2024

Thanks! I have confirmed this issue is fixed!
Thanks so much for always responding so quickly!

@broksonic21
Copy link

confirmed this fixed things for us too. thanks!

@ioquatix
Copy link
Member

Thanks for your confirmations!

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

No branches or pull requests

3 participants