autoop-module: no case-sensitive #457

New Issue

2026-10-05T04:22:36+02:00

adam commented

2026-10-05 04:22:36 +02:00

pls fix that the users hostmask is case-sensitiv

pls fix that the users hostmask is case-sensitiv

adam closed this issue

2026-10-05 04:22:36 +02:00

adam commented

2026-10-05 04:22:37 +02:00

Why should the users hostmask be considered case-sensitive when using the autoop module?

It makes no sense why case-sensitivity should matter - many IRC networks' vhosts/cloaks won't be 'case sensitive' enough to be an issue (in most networks), and rDNS lookups usually return as lowercase.

So back to my question, what's your reasoning here? (Also, use real english, not shorthand and crap.)

Why should the users hostmask be considered case-sensitive when using the autoop module? It makes no sense why case-sensitivity should matter - many IRC networks' vhosts/cloaks won't be 'case sensitive' enough to be an issue (in most networks), and rDNS lookups usually return as lowercase. So back to my question, what's your reasoning here? (Also, use _real english_, not shorthand and crap.)

adam commented

2026-10-05 04:22:37 +02:00

if the users ident changes from 'username@...' to 'Username@...' the users added host is not recognized anymore.

if the users ident changes from 'username@...' to 'Username@...' the users added host is not recognized anymore.

adam commented

2026-10-05 04:22:38 +02:00

Ah, so the issue is really that it should be case-insensitive. I think it is that, which version are you using? You can check with /msg *status version or running znc --version in terminal.

Ah, so the issue is really that it should be case-insensitive. I think it is that, which version are you using? You can check with `/msg *status version` or running `znc --version` in terminal.

adam commented

2026-10-05 04:22:39 +02:00

ZNC 1.5-nightly-20140724-8a5e7ef8

ZNC 1.5-nightly-20140724-8a5e7ef8

adam commented

2026-10-05 04:22:39 +02:00

The module seems to use case sensitive CString::WildCmp(). Sprinkling some AsLower() calls somewhere around HostMatches() should help.

The module seems to use case sensitive CString::WildCmp(). Sprinkling some AsLower() calls somewhere around HostMatches() should help.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: adam/znc#457