Skip to content

Conversation

@eaescob
Copy link

@eaescob eaescob commented Jan 16, 2025

Changed s_user.c to broadcast AWAY when setting/unsetting. Removed MODE +a/-a broadcast. This is so we can notify away message as part of supporting this capability.

Created a method in send.c - to send specified pattern to all channels the user is a part of, this helps for away-notify.

…ded for away-notify. Added definition for CAP_AWAY_NOTIFY and created broadcasting of AWAY on join and user setting or unsetting away.
@eaescob
Copy link
Author

eaescob commented Jan 16, 2025

Ugh. I clearly did not disable all the formatting settings. I can try to remove those changes.

@patrick-irc
Copy link
Member

patrick-irc commented Jan 18, 2025

Is AWAY still part of the burst? I think this was the biggest concern last time.

If not and +a is removed (also from burst), I think after netsplit we do not see if a user is away when doing WHOIS nick (not WHOIS nick nick) if he is not on common channels. I think this worked with +a.

@eaescob
Copy link
Author

eaescob commented Jan 19, 2025

I changed m_away to send AWAY to servers instead of MODE +a/-a.

@patrick-irc
Copy link
Member

As described above:

  1. User goes AWAY on server1
  2. server2 links to server1
  3. AWAY is not shown in WHOIS user on server2
WHOIS 000AAAAAA

00:32 -!- 000AAAAAA [patrick@127.0.0.1]
00:32 -!-  ircname  : x
00:32 -!-  server   : irc.localhost [Debian ircd default configuration]
00:32 -!- End of WHOIS

WHOIS 000AAAAAA 000AAAAAA

00:32 -!- 000AAAAAA [patrick@127.0.0.1]
00:32 -!-  ircname  : x
00:32 -!-  server   : irc.localhost [Debian ircd default configuration]
00:32 -!-  away     : away message
00:32 -!-           : is a Secure Connection (SSL/TLS)
00:32 -!-  idle     : 0 days 0 hours 4 mins 4 secs [signon: Sat Jan 25 00:28:23 2025]
00:32 -!- End of WHOIS

with +a first WHOIS shows at least:

00:43 -!-  away     : Gone, for more info use WHOIS 000AAAAAA 000AAAAAA

@eaescob
Copy link
Author

eaescob commented Jan 26, 2025 via email

@patrick-irc
Copy link
Member

Then I suggest to keep a and combine it with away-notify.

Adding AWAY to burst can cause serious issues as the burst now already takes 60 seconds and we do not know what happens if we add thousands of away messages.

@eaescob
Copy link
Author

eaescob commented Jan 27, 2025 via email

@eaescob
Copy link
Author

eaescob commented Jan 29, 2025

Changed PR to send MODE +a/-a on away/unaway and rely on the umode.

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.

2 participants