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

Add users to userlist on PRIVMSG event #646

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

voidlily
Copy link

A certain irc server doesn't respond to WHO properly, and we might have missed the join. This means that we might never actually know that that user is in the channel even if they are sending messages.

The best fix I could come up with client-side for this is to just add the user to the channel's userlist on a PRIVMSG event.

A certain irc server doesn't respond to WHO properly, and we might have missed
the join. This means that we might never actually know that that user is in the
channel even if they are sending messages.

The best fix I could come up with client-side for this is to just add the user
to the channel's userlist on a PRIVMSG event.
@4e554c4c
Copy link
Contributor

Unless if a channel is +n it is possible to send a privmsg to a channel that is not joined, so i don't think adding users directly to the channel list after privmsg is a great idea.

@voidlily
Copy link
Author

you know what, i honestly forgot that +n is a thing that you can turn off and that's a very good point to keep in mind.

@voidlily voidlily closed this Nov 22, 2024
@4e554c4c
Copy link
Contributor

i think this is a good thing to keep on the backlog (i know some ircds like twitch absolutely don't support WHO) but it would need some careful thinking

@voidlily
Copy link
Author

yeah the "some server" i was alluding to was twitch, wasn't sure how saying upfront that "yeah it's twitch" would be taken here, or if it would be "but it's not real irc we don't support that"

twitch does have a workaround with /CAP REQ :twitch.tv/membership but it's been weird in the exact formatting to get the server to accept it on this client (as compared to hexchat) - i've had the most success with /raw CAP REQ :twitch.tv/membership but it may have been without the colon, it's actually kind of hard to tell because i can't find like a raw response log that shows ping/pong and stuff like that anywhere to debug when i do get it right (sometimes i've even had issue with it not taking the command if i put it in the on_connect command list)

@voidlily
Copy link
Author

i can reopen this since can't easily like "turn it back into an issue" on github so it doesn't disappear into the aether, since it seems like something that may be worth thinking about in any case

@voidlily voidlily reopened this Nov 22, 2024
@4e554c4c
Copy link
Contributor

imo halloy should also have configurable cap requests like weechat

@casperstorm
Copy link
Member

imo halloy should also have configurable cap requests like weechat

Yeah i agree, and should be a easy one to implement.

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 this pull request may close these issues.

3 participants