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

Sometimes sales are missing from the bulk message, or are missed all together #3

Open
allisonking opened this issue Oct 26, 2022 · 0 comments
Labels
bug Something isn't working

Comments

@allisonking
Copy link
Owner

What's wrong

Occasionally, when a sale is reported to Discord, some items will be missing from the list. And there have also been instances where the entire sale is missed.

Context

It seems there is something fundamentally buggy with our architecture. There are a few checks going on to make sure we don't report manual inventory changes. We also check the total number of sales against what we last recorded as the total number of sales. I suspect one of these checks is causing things to go wonky, perhaps a race condition or something.

This is unfortunately difficult to debug. At first, we suspected we were sending too many Discord messages and some were getting dropped. 62f2757 changed the architecture to send all sale reports as one message instead of each sale being a message in an attempt to fix this problem. However, the problem still persists, so it does not appear to be the case that Discord was dropping messages.

Suggested path forward

Since this is pretty difficult to debug, I think our first step is to add more logging, perhaps to another file that is easier to consult than the current output log file. We should also take a harder look at our system architecture to see where it may be possible that messages are getting dropped.

@allisonking allisonking added the bug Something isn't working label Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant