View Issue Details

IDProjectCategoryView StatusLast Update
0004232unrealircdpublic2014-05-30 19:32
ReporterTechman Assigned Tosyzop  
Status resolvedResolutionfixed 
PlatformLinuxOSUbuntuOS Version12.04 x86_64
Product Version3.2.10.1 
Fixed in Version3.4-alpha1 
Summary0004232: Change "Rejoining because of user@host change" text on part message to "Changing Host"
DescriptionChange the text used on the set::allow-userhost-change setting when set to force-rejoin. When a user changes his/her host, the text will be changed from what it is now to just "Changing Host". Also if possible, the client could be shown as quitting instead of parting, although I don't really care about that feature (I just wanted to mention it).
TagsNo tags attached.
3rd party modulesm_sanick textban



2013-08-11 03:32

reporter   ~0017744

I don't see a need to change the text.

Could you provide justification for such a change?

How is sending a QUIT instead of a PART better?
Could sending a QUIT instead possibly be worse?
Could a QUIT vs PART cause issues or race conditions?

How will QUIT be handled between servers?
Will a server need to QUIT a client and reintroduce the client as a fresh connection?
How would s2s handle user timestamps if the user is QUIT and reintroduced?


2013-08-11 04:09

reporter   ~0017746

You don't have to change the part sequence to a quit sequence, because I said that wasn't really needed but I would mention it.

I just want the text changed from what it is now to "Changing Host", something that is a lot more simple and shorter than saying rejoining because of user@host change. You guys told me that if I change this on my own that you guys won't support me anymore, so this is the only way to get it officially changed.


2013-08-11 17:44

reporter   ~0017747

Other than "It's shorter" I fail to see any real reasoning. I cannot recall anyone else who has requested something shorter either here, on IRC, or the forums.


2013-08-14 09:48

administrator   ~0017751

It seems nenolod changed (in Unreal 3.4.x devel branch) PART/JOIN to QUIT/JOIN in -- personally I would rather have preferred to see it as an option, though. He just ripped the PART/JOIN thing out :p.

As for changing the wording. In the same commit, nenolod changed it to 'Changing host'.

I think the original message was more clear, but the new message is smaller and prettier. Can't say I care much about this :P


2013-08-25 01:23

reporter   ~0017759

I don't see a need to change either. Saying "simple and shorter" implies that the current method is long and complex which it isn't. IMVHO, this is another minor change that yields little or no advantage.

As far as quit vs part, I feel part is better. Part is all that is needed to keep channels and other clients in sync. Sending a quit could have adverse effects on client bots and services as they may require the user log in again "because they quit".

I see the reason for the change was because other ircds do it, that doesn't make it "better" or more correct. Unreal isn't other ircds and in fact, if all ircds did the exact same things, we would only need 1. I think it's bad form to do something based solely on the fact that others do it. We aren't sheep so we don't need to jump off the cliff just because others have.


2013-08-25 09:46

administrator   ~0017760

I totally agree, especially on the 'other ircds do it this way so we must do it too' criticism.
I've seen a few changes or suggestions about making things 'like in other ircds', to me that's fine if it's an improvement, but to those people I say: remember that sometimes.. yes sometimes.. we think a little about things too! and then we MAY just decide different on things. There are often pro's and con's on something, and we may just weigh them different than other ircds do and make another decision than them.

Now, this doesn't have that much to do with the original report anymore, as said I'm fine with a text change (or not) ;p.


2014-05-30 19:31

administrator   ~0018149

It's "Changing host" in 3.4.x now.

I've undone nenolod's QUIT+JOIN thing, don't like it. It's PART+JOIN like in 3.2.x. now.

Issue History

Date Modified Username Field Change
2013-08-10 04:49 Techman New Issue
2013-08-11 03:32 Stealth Note Added: 0017744
2013-08-11 03:32 Stealth Assigned To => Stealth
2013-08-11 03:32 Stealth Status new => feedback
2013-08-11 04:09 Techman Note Added: 0017746
2013-08-11 17:44 Stealth Note Added: 0017747
2013-08-14 09:48 syzop Note Added: 0017751
2013-08-25 01:23 katsklaw Note Added: 0017759
2013-08-25 09:46 syzop Note Added: 0017760
2014-03-14 01:14 peterkingalexander Issue cloned: 0004286
2014-05-30 19:31 syzop Note Added: 0018149
2014-05-30 19:31 syzop Status feedback => resolved
2014-05-30 19:31 syzop Fixed in Version => 3.4-alpha1
2014-05-30 19:31 syzop Resolution open => fixed
2014-05-30 19:31 syzop Assigned To Stealth => syzop