-
Notifications
You must be signed in to change notification settings - Fork 130
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
Upgrade to aiocoap 0.4b3 #274
Conversation
The other flake8 errors will be squashed once we migrate this repo to black once this is merged. |
Both initial and secondary (without key) seems to work as expected 👍 with my limited test (connect)
I don't actually have something paired to it for further testing (the blinds are elsewhere now) |
async def _get_protocol(self): | ||
"""Get the protocol for the request.""" | ||
if self._protocol is None: | ||
self._protocol = asyncio.Task(Context.create_client_context( | ||
loop=self._loop)) | ||
self._protocol = asyncio.create_task(Context.create_client_context()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
line too long (81 > 79 characters)
This upgrades pytradfri to use aiocoap 0.4b3 and migrates credential management to the new approach.
I don't have a Tradfri gateway here so I only got as far as that aiocoap was able to find the credentials. Was not able to verify.
Fixes #185
Can be tested using:
This is a breaking change. Async API Factory now needs to be set using
await APIFActory.init(…)