UnrealIRCd Bug Tracker - unreal - ISSUES (Unreal4 QA Needs Testing)

0003450: Filters (m_filter.so and m_filter_pcre.so ) aren't removed with rehash

When removing or editing a filter in the inspircd.conf, after rehash the old filters remain (/stats s). My experience was also that due to this, my replacement filter (as simple as ^querty$) was not working.

0003437: localhost should be cloaked

Brain insists it's invalid, but I don't like people on my network knowing someone is connecting from a localhost... This creates issues like "zomg, I want access to use IRC too!"

Unlike Unreal 3.*, 4 should probably cloak the actual localhost IP in instead of "localhost" itself.

0003439: m_hostchange.so tweak

When using the "set" action, have a nick variable for the user's nick...

Something like:

%n would become the users nick (of course without non-RFC characters), the result should be "users.SomeNick.example.com" (if the user is connecting with "SomeNick")

0003513: Extend chanrec::IsBanned (so we can have extbans)

The chanrec::IsBanned provides only simplistic implementation of ban checking, eg, if a n!u@h mask matches in the channel. In order to implement extended bans of any form, IsBanned will need at least two things:

- Able to identify the type of action taken that is causing a ban check (eg: JOIN, NICK, PRIVMSG).
- Able to call it module code before doing a normal check of a ban entry. Includes allowing the module to say "don't do a normal mask check".

The first issue, I've considered:
- Unreal 3's style: use named constants BANCHK_JOIN, etc. Advantage: no string compares. Disadvantage: modules have to fight over unique values, or we have to make a provider for it. Probably the best option, performance-wise.
- Something closer to how insp does assorted things, namely: use text strings. Advantage: no fighting over uniqueness, if they just use their cmd name. Disadvantage: string compares, with all the slowdowns that can involve . Especially if some extbans want to check against several different command names (eg a ~q would need to trap PRIVMSG and NOTICE - double whammy, though we can avoid it with using non-command names like 'MESSAGE').
- Other ideads?

For the second issue:

Either built-in check for extended ban syntax ('~', something, ':'), or a hook that is triggered for every ban checked (that way module can provide the extban system, and other modules can do things to a specific ban check that don't strictly relate to extended bans). The latter is probably preferred.

0003519: Implement @merge config directive

When the config transformations meet a @merge config directive on form:

@merge block-name;

it will merge all blocks with this block-name into one.

for instance:

features/default.conf:

@merge set;

set { // some defaults; };

unrealircd.conf:

include "features/default.conf";

set { // override some of the defaults };

to the IRCd the set {} block will look as if it is one tag.

Conflict resolution in the case of two matching key's (key "value"; ) in block, is the the latest setting is the one that wins.

0003521: configure does not pull epoll support properly.

[cvm@camel24:~/hobby/unreal4]$ strings /lib/libc.so.6 | grep "GNU C Library"
GNU C Library stable release version 2.5, by Roland McGrath et al.
[cvm@camel24:~/hobby/unreal4]$ uname -r
2.6.20-1.2952.fc6

Checking if epoll exists... libc too old: 5... libc built against older kernel 9... no

0003515: Added ban-server {} configuration

- Added ban-server {} support - disconnects a server if it dares to
introduce a banned server to the network. Conf format:
ban-server { mask "*mask"; reason "reason"; };

0003486: Implemented ENCAP proposal

http://dev.unrealircd.com/unreal4:development:encap [^]

Basically, we need the ability to modularilly to do s2s queries - one server querying another and send back a response one to one.

This is needed for example for RPING.

We have been working together with Insp to provide a similar API so modules that uses this will work on future Insp 1.2.

This will need a LOT of QA

0003463: Implement parser for U3 configuration format

As above.

0003496: SSL ports in 005 numeric

The 005 numeric displays 'SSL=' if no SSL ports have been defined when any of the SSL modules (openssl and gnutls) have been loaded. My suggestion would be to hide this unless SSL ports have been defined within any of the client 'bind' config blocks.

0003465: Link U3 parser to existing InspIRCd configuration code

Make it possible to access the U3 format through the existing ConfigReader methods.