View Issue Details

IDProjectCategoryView StatusLast Update
0004413unrealircdpublic2015-10-28 11:24
Reportersyzop Assigned Tosyzop  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionno change required 
Product Version4.0.0-rc1 
Summary0004413: server wildcard no longer working
DescriptionDon't know if it's an intentional change, but I noticed that /SQUIT serv* isn't working anymore, so wildcard expansion.

Maybe it's ok, maybe not, but isn't documented anywhere nor have I thought about the implications.
TagsNo tags attached.
3rd party modules

Activities

dboyz

2015-09-29 18:52

reporter   ~0018702

Hi,

Unreal3.2 uses next_client() which enables wildcard usage by using the match() function.
See: https://github.com/unrealircd/unrealircd/blob/master/src/modules/m_squit.c#L109-L117

next_client() function seems to be depreciated in Unreal3.4 - although the code still exists, it is not being called anywhere.

Unreal3.4 currently uses find_server_quickx for SQUIT which does not allow wildcard usages. A solution for this is to revert to next_client() or switch to find_server_quick_search() or find_match_server().

There is a macro called find_server_quick() which maps to find_server_quickx(), used intensively elsewhere except in SQUIT. And what I discover next is interesting (see 0004421).

syzop

2015-10-28 11:24

administrator   ~0018802

It was broken in hunt_server as well. I rewrote the function https://github.com/unrealircd/unrealircd/commit/d81c52c5aa872d6234dd164903b8c2eb93ed1956

BUT.. I left wildcard support out. I think we can do without, then we don't have to deal with ambigious wildcard shit and depend on random order who wins (eg /squit *.net -- if it would have used hunt_server, that is)

Issue History

Date Modified Username Field Change
2015-09-07 11:55 syzop New Issue
2015-09-29 18:52 dboyz Note Added: 0018702
2015-10-28 11:24 syzop Note Added: 0018802
2015-10-28 11:24 syzop Status new => closed
2015-10-28 11:24 syzop Assigned To => syzop
2015-10-28 11:24 syzop Resolution open => no change required