search

ComputerScienceHouse / letmein2

A second wack at LetMeIn. It came to me in a dream!
MIT License
6 stars 5 forks source link

Add slackbot support #27

Closed WillNilges closed 2 years ago

WillNilges commented 2 years ago

The code is kind of a mess, but this adds slackbot support so that you'll know who's requesting to be let in.

WillNilges commented 2 years ago

Via some other fuckery I'm doing on this branch for some reason (trying to clean up the code >_>), this introduces a regression where (once again) if you have multiple requests and timeout on one of them, something sends an ACK message to the ack topic and wrongfully "answers" other pending requests.

Guh.

WillNilges commented 2 years ago

Hmmm. Am I going to need to keep track of active sessions... I think so. Maybe this bug has always been here, and I just never noticed? I could have sworn I fixed this though.

WillNilges commented 2 years ago

Ok, I think I fixed that bug, but there's some issues with the client that are unrelated to these changes that I noticed. Seems like if one request times out, they'll all time out. This is probably already known, and I think I can fix this by mucking with the MQTT protocol.