View Issue Details

IDProjectCategoryView StatusLast Update
0003716unrealircdpublic2015-10-01 18:11
Reporternate Assigned Tosyzop  
PrioritynormalSeveritytweakReproducibilityN/A
Status closedResolutionno change required 
Summary0003716: nick greater than NICKLEN error
DescriptionImplement either a soft notice or a block (perhaps make it optional at ./Config -advanced what to do) of /nick when the nickname length equals more than the NICKLEN

Default implementation I would say would be a non-block, in the form of a snotice to the user to let them know their nick was trimmed due to nicklength overage?

Opinions, ideas?
3rd party modules

Activities

syzop

2013-01-09 09:56

administrator   ~0017307

Do other IRCds do this? If so, might be a good idea.
But don't block.

dboyz

2015-09-30 06:39

reporter   ~0018707

Logically long newnick should not be accepted. However, surprise surprise - only inspircd sends ERR_ERRONEUSNICKNAME and deny the nickchange. Other ircds - bahamut, charybdis, nefarious, mammon behaves the same way like unreal, truncates the newnick.

syzop

2015-10-01 18:11

administrator   ~0018722

Thanks for looking into this.

I think we can keep truncating then.

Issue History

Date Modified Username Field Change
2008-08-09 23:20 nate New Issue
2008-08-09 23:20 nate Status new => assigned
2008-08-09 23:20 nate Assigned To => nate
2008-08-09 23:20 nate QA => No need for QA
2008-08-09 23:20 nate U4: Need for upstream patch => No need for upstream InspIRCd patch
2008-08-09 23:20 nate U4: Upstream notification of bug => Not decided
2008-08-09 23:20 nate U4: Contributor working on this => None
2013-01-09 09:56 syzop Note Added: 0017307
2013-01-09 09:56 syzop Assigned To nate =>
2013-01-09 09:56 syzop Status assigned => feedback
2013-01-09 09:56 syzop Product Version 3.3-alpha0 =>
2015-09-30 06:39 dboyz Note Added: 0018707
2015-10-01 18:11 syzop Note Added: 0018722
2015-10-01 18:11 syzop Status feedback => closed
2015-10-01 18:11 syzop Assigned To => syzop
2015-10-01 18:11 syzop Resolution open => no change required