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

rtm.start depreciated starting 2022/09/20 #304

Open
hartjac23 opened this issue May 10, 2022 · 7 comments
Open

rtm.start depreciated starting 2022/09/20 #304

hartjac23 opened this issue May 10, 2022 · 7 comments

Comments

@hartjac23
Copy link

It looks like the rtm.start API method is depreciated in september of 2022, and will start returning a shortened response identical to rtm.connect. Are there plans to update the SlackClient to move over to rtm.connect and handle the new truncated response?

@kanadagermane
Copy link

Hi,

maybe its just me, but I cannot get the /api/rtm.start method working at all.
It requires the scope "rtm.stream" and I cannot add this scope to my user token?

@iclaxton
Copy link

It looks like this lib no longer works. Any chance of a hotfix?

@glenwperry
Copy link

glenwperry commented Sep 28, 2022

Also hoping for this ^ . Didn't realize it was using rtm.start (api changed yesterday) - but it indeed appears to.

@Inumedia
Copy link
Owner

I'm not opposed to accepting a PR that would resolve this, but I don't personally have any reason to fix it. I've also got a lot of things happening in my personal life, so this library isn't a priority for me at this time.

@vedxyz
Copy link

vedxyz commented Sep 30, 2022

If you do not need the Slack Real Time Messaging API specifically, I'd suggest switching over to soxtoby/SlackNet.

Also:

1- You probably shouldn't be using a socket-based API if the Web API serves your purposes.
2- You probably should be using the new Socket Mode API if you really need to keep track of events through sockets.

@iclaxton
Copy link

Hi @Inumedia, I totally understand and will help unless anyone else is already on it. Sending my best regards and thanks for all your work on this lib to date.

@ceciliachan1979
Copy link

I think I have some success on getting it to work "somehow".

All you needed to do is to comment out the body of SlackAPI.SlackClient.Connected here

The symptom is that this method is throwing a NullReferenceException when accessing loginDetails.user and the rest go south.

The cause is that rtm.start will no longer provide those values (assuming it did, I can't validate because the documentation no longer show the fields before the deprecation), so access them will fail.

Apparently, commenting out the body will lead to the fact that these properties are no longer initialized, so there might be downstream errors.

But something is better than nothing. Now I got the messages from the onMessage method.

To test this, make sure you create a classic slack app, the new granular permission version do not support rtm.

Long term, I agree with @Vedattt, socket mode is the way to go. I don't know if I want to take the dependency on aspnet.core though, for me personally, the whole point of using a slack frontend is to avoid having to manage a listening socket myself.

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

No branches or pull requests

7 participants