FS#80059 - [unrealircd] geoip_classic module missing

Attached to Project: Arch Linux
Opened by Juest Zungo (Juest) - Monday, 23 October 2023, 01:11 GMT
Last edited by Buggy McBugFace (bugbot) - Saturday, 25 November 2023, 20:20 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Sergej Pupykin (sergej)
Architecture All
Severity Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
out of the box configurations cannot run the server without modifying modules.default.conf which is not supposed to be modified

Additional info:
* package version(s)
* config and/or log files etc.
* link to upstream bug report, if any

Version: 6.1.2.3-1
[error] /etc/unrealircd/modules.default.conf:299: loadmodule geoip_classic: failed to load: Cannot open module file: No such file or directory

Steps to reproduce:

attempt to run the server interactively from the ircd user after editing unrealircd.conf
This task depends upon

Closed by  Buggy McBugFace (bugbot)
Saturday, 25 November 2023, 20:20 GMT
Reason for closing:  Moved
Additional comments about closing:  https://gitlab.archlinux.org/archlinux/p ackaging/packages/unrealircd/issues/1
Comment by Toolybird (Toolybird) - Monday, 23 October 2023, 05:08 GMT
This is expected. From the wiki [1] "There is a default configuration but it does not work out of the box." The only possible improvement here would be to add the additional .conf file(s) that need editing to the backup array.

[1] https://wiki.archlinux.org/title/UnrealIRCd#Configuring_%28mandatory%29
Comment by Juest Zungo (Juest) - Monday, 23 October 2023, 15:16 GMT
well, why geoip_classic is not built in first place if upstream builds it?
Comment by Juest Zungo (Juest) - Monday, 23 October 2023, 15:37 GMT
the problem here is that a module from upstream is not included, not really a config issue, sorry for making it looking that way
Comment by loqs (loqs) - Monday, 23 October 2023, 16:36 GMT
That would require adding geoip as a dependency or using the bundled copy of the library. geoip supports the legacy format that Maxmind no longer supply database files in. Maxmind's current format is supported by libmaxminddb.
When --enable-geoip-classic=yes is not passed to ./configure the geoip_classic module is not built but it is still referenced in modules.default.conf.

[1] https://dev.maxmind.com/geoip/release-notes/2022#geoip-legacy-databases-have-been-retired
Comment by Juest Zungo (Juest) - Monday, 23 October 2023, 16:49 GMT
I understand, but the built-in geoip classic comes with automatic database updates provided by upstream, without user maintenance.
I see, so upstream configuration expects the user to run with geoip_classic built.
upstream has a configure script that asks questions and the question about geoip support defaults to building geoip_classic, between geoip_classic, none and a user provided libmaxminddb powered module
Comment by loqs (loqs) - Monday, 23 October 2023, 17:22 GMT
Looking at what other distributions do Fedora chose to remove geoip_classic [1], Debian does not package unrealircd, gentoo enables building libmaxminddb.

[1] https://src.fedoraproject.org/rpms/unrealircd/blob/rawhide/f/unrealircd-6.1.0-geoip.patch
Comment by Juest Zungo (Juest) - Monday, 23 October 2023, 18:26 GMT
right, unrealircd shouldn't be packaged in first place, upstream doesn't recommend it

Loading...