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

Error with the scout add-on in Heroku #522

Closed
danitico opened this issue May 21, 2020 · 2 comments · Fixed by #523
Closed

Error with the scout add-on in Heroku #522

danitico opened this issue May 21, 2020 · 2 comments · Fixed by #523

Comments

@danitico
Copy link

With version 2.14.1 of this package, I encounter the following error:
` File "/code/nluproxy/init.py", line 3, in

from .celery import app as celery_app

File "/code/nluproxy/celery.py", line 12, in

scout_apm.celery.install(app)

File "/usr/local/lib/python3.7/site-packages/scout_apm/celery.py", line 58, in install

installed = scout_apm.core.install()

File "/usr/local/lib/python3.7/site-packages/scout_apm/compat.py", line 132, in wrapper

return func(*args, **kwargs)

File "/usr/local/lib/python3.7/site-packages/scout_apm/core/init.py", line 46, in install

report_app_metadata()

File "/usr/local/lib/python3.7/site-packages/scout_apm/core/metadata.py", line 17, in report_app_metadata

event_value=get_metadata(),

File "/usr/local/lib/python3.7/site-packages/scout_apm/core/metadata.py", line 37, in get_metadata

"libraries": get_python_packages_versions(),

File "/usr/local/lib/python3.7/site-packages/scout_apm/core/metadata.py", line 60, in get_python_packages_versions

for distribution in distributions()

TypeError: '<' not supported between instances of 'NoneType' and 'str'

`

@adamchainz
Copy link
Contributor

Fixed in #523, released in version 2.14.2. Please upgrade and let me know if it still doesn't work.

The issue was with an installed package not having a name or version. We now gracefully handle such packages. If this started happening because you upgraded a package, it could be useful to know which package it is, at least to try fix it there.

@danitico
Copy link
Author

It works @adamchainz 🎉

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.

2 participants