Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 71605 - [relay-ctrl new feature] made relay-ctrl-check read additional env settings from a file
Summary: [relay-ctrl new feature] made relay-ctrl-check read additional env settings f...
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Server (show other bugs)
Hardware: All All
: Normal enhancement (vote)
Assignee: Qmail Team (OBSOLETE)
URL:
Whiteboard:
Keywords: Inclusion
Depends on:
Blocks:
 
Reported: 2004-11-17 14:49 UTC by Perolo Silantico
Modified: 2006-11-14 04:04 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
relay-ctrl-3.1.1-r2.package.diff (relay-ctrl-3.1.1-r2.diff,7.25 KB, patch)
2004-11-17 14:51 UTC, Perolo Silantico
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Perolo Silantico 2004-11-17 14:49:09 UTC
When using relay-ctrl with qmail-dk (domain keys module for qmail) I run into trouble to set additional environment data (fe. DKSIGN, or NOSPP for qmail-scanner) only for authenticated users. Although I scrutinized the sources I did not find how relay-ctr-check or relay-ctrl-allow may add a standard set of additional settings to the environment. So I altered relay-ctrl-check to do so.

If "RELAY_CTRL_RELAYCLIENT_ENV" is set to a file name (fe: RELAY_CTRL_RELAYCLIENT_ENV='/var/spool/relay-ctrl/client.env') then this file is being read by relay-ctrl-check if the proper IP file can be found. Hence only authorized IPs (clients) have these values in their environment. Any previous value is overwritten!
Comment 1 Perolo Silantico 2004-11-17 14:51:23 UTC
Created attachment 44189 [details, diff]
relay-ctrl-3.1.1-r2.package.diff

a patch to relay-ctrl-3.1.1-r2.ebuild that includes a new enhancement to
relay-ctrl-check. Necessary additional files are created and digests modified.

Apply the patch within the ebuild package dir.
Comment 2 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2006-05-30 13:43:51 UTC
Moving this to qmail-bugs proper, I'll deal with it a bit later.
Comment 3 Luca Lesinigo 2006-09-24 14:25:49 UTC
I see you already have a patch, but maybe this could be useful.

relay-ctrl-check already does what you want: it will read the contents of the 'ip file' in /var/spool/relay-ctrl/allow/W.X.Y.Z and parse it looking for environment variables (NULL-terminated list, eg: "A=one\0B=two\0" and so on).

relay-ctrl-allow use this feature to  add USER and DOMAIN variables, per-ip.
Comment 4 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2006-11-14 04:04:54 UTC
since luckyluke points out that relay-ctrl already does what you want, I'm closing this bug.