8000 fix: `asyncio` exception by bieniu · Pull Request #7 · pysnmp/pysnmp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix: asyncio exception #7

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 26, 2022
Merged

fix: asyncio exception #7

merged 1 commit into from
May 26, 2022

Conversation

bieniu
Copy link
@bieniu bieniu commented May 9, 2022

When using asyncio an exception is raised:

Error doing job: Task exception was never retrieved
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/pysnmp/carrier/asyncio/dispatch.py", line 56, in handle_timeout
yield asyncio.From(asyncio.sleep(self.getTimerResolution()))
AttributeError: module 'asyncio' has no attribute 'From'

This PR fixes this.

Related to: home-assistant/core#71368

@bieniu
Copy link
Author
bieniu commented May 15, 2022

I have read the CLA Document and I hereby sign the CLA

@bieniu
Copy link
Author
bieniu commented May 15, 2022

I have read the DCO Document and I hereby sign the DCO

@bieniu
Copy link
Author
bieniu commented May 15, 2022

recheck

@bieniu bieniu mentioned this pull request May 15, 2022
Merged
22 tasks
@ryanfaircloth ryanfaircloth changed the title Fix asyncio exception fix: asyncio exception May 26, 2022
@ryanfaircloth ryanfaircloth merged commit aa75347 into pysnmp:main May 26, 2022
@bieniu bieniu deleted the fix-asyncio branch May 26, 2022 14:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ryanfaircloth ryanfaircloth ryanfaircloth approved these changes

@lzzsalvo lzzsalvo lzzsalvo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bieniu @ryanfaircloth @lzzsalvo
0