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

My own messages are shown twice #128

Closed
ngortheone opened this issue May 21, 2020 · 8 comments · Fixed by #135
Closed

My own messages are shown twice #128

ngortheone opened this issue May 21, 2020 · 8 comments · Fixed by #135

Comments

@ngortheone
Copy link

Hi. First off thanks for the great app!

I use Quassel IRC and when I send a message a split second later the same message appears again. As if I sent it twice.

Others do not see doubled messages, even I don't see it in native slack client. So it is probably something between Quassel and irc-slack that makes Qassel show my own message 2 times.

@chex3
Copy link

chex3 commented May 21, 2020

I have this issue, with irssi, also.

  • Stefan

@ajgon
Copy link

ajgon commented May 23, 2020

Same problem with thelounge.

@janza
Copy link
Collaborator

janza commented Jun 11, 2020

Hi, are you using Slack App token or legacy tokens for authentication with slack?

@insomniacslk
Copy link
Owner

Now I'm seeing this too, using a refreshed token/cookie on an MFA-enabled account. @ngortheone @chex3 @ajgon are you still seeing this? And as Josip was also asking, which type of token are you using?

@insomniacslk
Copy link
Owner

Btw I plan to fix this over the weekend

@insomniacslk
Copy link
Owner

quick repro: this happens only with token+cookie, not with legacy tokens

@chex3
Copy link

chex3 commented Jul 17, 2020

Hi, I am definitely still seeing this issue, and yes, I use token+cookie.

  • Stefan

example:

11:10:58 chex38 | woo hooo
11:10:59 U0SN7DR1S | woo hooo
11:11:07 chex38 | do you see messages double??
11:11:08 U0SN7DR1S | do you see messages double??

insomniacslk added a commit that referenced this issue Jul 17, 2020
Fixes #128

Signed-off-by: Andrea Barberio <insomniac@slackware.it>
@insomniacslk
Copy link
Owner

The reason is that Slack sends an event with your own message when using the new style tokens.

This should be fixed in #135 . Can people on this thread test it too before we merge it?

insomniacslk added a commit that referenced this issue Jul 17, 2020
Fixes #128

Signed-off-by: Andrea Barberio <insomniac@slackware.it>
insomniacslk added a commit that referenced this issue Jul 17, 2020
Fixes #128

Signed-off-by: Andrea Barberio <insomniac@slackware.it>
insomniacslk added a commit that referenced this issue Jul 17, 2020
Fixes #128

Signed-off-by: Andrea Barberio <insomniac@slackware.it>
insomniacslk added a commit that referenced this issue Jul 17, 2020
Fixes #128

Signed-off-by: Andrea Barberio <insomniac@slackware.it>
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

Successfully merging a pull request may close this issue.

5 participants