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

GELFRabbitHandler not imported in python new versions. #140

Open
PabloEmidio opened this issue Aug 10, 2022 · 5 comments · May be fixed by #143
Open

GELFRabbitHandler not imported in python new versions. #140

PabloEmidio opened this issue Aug 10, 2022 · 5 comments · May be fixed by #143

Comments

@PabloEmidio
Copy link

PabloEmidio commented Aug 10, 2022

When use python3.9.13 I ran into the following error:
AttributeError: module 'graypy' has no attribute 'GELFRabbitHandler

Conditions:
graypy==2.1.0
amqplib==1.0.2

@PabloEmidio
Copy link
Author

Hey! Is this lib still suported? Debugging the error, I found out that the problem is the amqplib (last update was 7 years ago) and it's being suppressed by the ImportError. But great news, amqlib was forked and now it's the official suport of Celery organization in this repository.

try:
from graypy.rabbitmq import GELFRabbitHandler, ExcludeFilter
except ImportError:
pass # amqplib is probably not installed

It's time to consider migrate. Could we discuss it?

@PabloEmidio PabloEmidio changed the title GELFRabbitHandler not imported on python3.9.13 GELFRabbitHandler not imported in python new versions. Jan 15, 2023
@Petchesi-Iulian
Copy link

I also encounter this issue with
graypy==2.1.0
amqplib==1.0.2

@severb
Copy link
Owner

severb commented Jul 19, 2023

Would either of you be interested in opening a pull request to fix this issue?

@Felix2048 Felix2048 linked a pull request Oct 24, 2023 that will close this issue
@severb
Copy link
Owner

severb commented Jul 10, 2024

Was anyone able to test the fix?

I don't have an active development setup for graypy right now but I'm more than happy to merge the fix if others can confirm it.

@wolbernd
Copy link

@severb Sorry for the late reply. I applied the PR from @Felix2048 in our test environment and everything works fine.

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 a pull request may close this issue.

4 participants