Skip to content
This repository has been archived by the owner on Jun 18, 2024. It is now read-only.

backlog often appear duplicated on the IRC side. #301

Open
terceiro opened this issue Apr 30, 2021 · 10 comments
Open

backlog often appear duplicated on the IRC side. #301

terceiro opened this issue Apr 30, 2021 · 10 comments

Comments

@terceiro
Copy link
Contributor

When I open my IRC client,I will often see the full backlokg appear twice, like this:

msg1
msg2
msg3
msg1
msg2
msg3

for example I am connected to 3 workspaces, and this morning I got this in all of them.

@ltworf
Copy link
Owner

ltworf commented Apr 30, 2021

Uhm I haven't encountered this.

Basically it calls an API to get history from a timestamp. Are the messages in threads maybe?

@terceiro
Copy link
Contributor Author

terceiro commented May 1, 2021 via email

@ltworf
Copy link
Owner

ltworf commented May 2, 2021

Could you check what the history api call is returning? I can't say I've seen the issue. In my experience they even remove all edits and stuff and just send the final message including all the reactions.

@terceiro
Copy link
Contributor Author

terceiro commented May 3, 2021 via email

@ltworf
Copy link
Owner

ltworf commented Jun 20, 2022

Is this still an issue?

@terceiro
Copy link
Contributor Author

very much, happens to me all the time

@terceiro
Copy link
Contributor Author

to be fair, I hadn't updated localslackirc in a while (was running 148dab8). I just pulled the latest master (9582618) and will let you know

@terceiro
Copy link
Contributor Author

very much, happens to me all the time

I mean, it happens very frequently, but not all the time or on all channels.

@ltworf
Copy link
Owner

ltworf commented Jun 22, 2022

Hm ok. I did have the issue as well but I haven't encountered it for a while. But there are still issues if the history is too long, because it blocks for too long to fetch it, in those cases I remove the state file and give up on the history.

@ltworf
Copy link
Owner

ltworf commented Sep 25, 2022

This should be fixed now, unless you kill the process without letting it store the state.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants