-
Notifications
You must be signed in to change notification settings - Fork 70
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
Connection timed out #294
Comments
Based on a note in #290 (reply in thread) and a quick review of Skype for Web's activity, changes seem to be:
I've pushed a minimal fix to branch You can install it via the repo with |
I confirm with this sending chat message to a contact or a group/thread is also working. I will report if anything happens before we migrated to teams. Thanks. |
Thanks for the update.
Code sample which was working fine recently:
|
If they've tightened the rate limits as part of this migration then there's little we can do about it. Do you get similarly limited using Skype for Web? |
In Skype for Web I can send how many messages I want without any rate limit. |
Try again with the latest commit. If you're still being unreasonably limited, take a look at the headers and body of the failing request in SkPy (with |
It helped to increase the number of messages to 7 before getting rate limited. |
If (part of) the fix is to change the server to a Teams one, does that mean this can be adapted for Teams in the future (eg a TmPy)? |
I encountered an issue during initialization. After running:
I received the following error:
Could you please help me troubleshoot this? Thank you! |
I encountered the exact same error as described above. Has anyone found a solution or workaround for this? |
From watching Skype for Web, it doesn't look like it listens for events any more? If you receive a message while a different chat is selected, the message doesn't come through. After a while (e.g. if you leave the tab in the background and later return to it), the client refreshes the conversation list, notices the chat has been updated, then fetches its messages again to get the new one. This probably means the long-polling call we use for listening to events doesn't exist in the Teams-backed API (this also lines up with the 501 error, which corresponds to HTTP 501 Not Implemented), so You can do something similar to Skype for Web and manually poll for updates via |
@Terrance also I get Line 497 in 5f75b96
It happens when I try to do
So, looks like that I have too much conversations to get all chats objects to a list. And I can't set limits of requests, because it is part of your lib (just after fork). Do you have any ideas of way out? |
This comment has been minimized.
This comment has been minimized.
Things to try:
|
What do you think about re-writing the loop to pull recent chats and iterate through them as if it was an event like how the loop worked with the proper rate limiting. Would this be difficult? I may give it a try, but my code is nowhere near production grade. |
As a reminder, this library is not production-grade. The current implementation of sync-state URLs (so that when you call e.g. Pull requests welcome for an alternate event loop implementation. |
Before we start...
Summary
Since march 10 I am getting connection timed out. It seems sk.user didn't get the token and kept requesting until hit the limit
Code sample
Code output
Explain your code
Simple test code
SkPy version
Latest
Python version
3.10
Anything else?
No response
The text was updated successfully, but these errors were encountered: