-
Notifications
You must be signed in to change notification settings - Fork 835
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
Use rtm.connect
method over rtm.start
to get websocket URL.
#190
Comments
This was referenced Jun 26, 2017
4 tasks
9 tasks
2 tasks
Roach
added a commit
that referenced
this issue
Aug 23, 2017
* Added rtm.connect support * updated docs * updated gitignore * Uncommented lines which were accidentally left commented * Updated docs * Removed ignored doc source files * Fixed typo in method docs * fixed line length * updated rim start tests * regenerated docs
c-goosen
pushed a commit
to c-goosen/python-slackclient
that referenced
this issue
Jun 18, 2019
* Added rtm.connect support * updated docs * updated gitignore * Uncommented lines which were accidentally left commented * Updated docs * Removed ignored doc source files * Fixed typo in method docs * fixed line length * updated rim start tests * regenerated docs
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
On very large Slack teams, the
rtm.start
payload can be too large to deliver before a timeout (server timeouts would return response 500, but also the websocket URL itself is short-lived so the timeout could be from the time it takes for the client to read the response).The Slack platform has a new method called
rtm.connect
which is much smaller and faster to load. It also would provide the websocket URL.I think this can be implemented as a swap of one for the other because the
rtm.start
data is not being returned to the user anyway (see #181), so that means there's no breaking API change by not having all the same data. The point about that data being valuable is valid though, so I propose that the default would be to usertm.connect
(better for most people anyway) and there is an option for usingrtm.start
, and in both cases the data is returned.What type of issue is this? (place an
x
in one of the[ ]
)Requirements (place an
x
in each of the[ ]
)The text was updated successfully, but these errors were encountered: