[pmg-devel] [PATCH api 1/2] auth: set PAM context to 'proxmox-mailgateway-auth'

[pmg-devel] [PATCH api 1/2] auth: set PAM context to 'proxmox-mailgateway-auth'

[Wolfgang Bumiller]

This allows configuring PAM authentication for PMG via
/etc/pam.d/proxmox-mailgateway-auth

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
---
 src/PMG/AccessControl.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PMG/AccessControl.pm b/src/PMG/AccessControl.pm
index b093666..7926d4b 100644
--- a/src/PMG/AccessControl.pm
+++ b/src/PMG/AccessControl.pm
@@ -127,7 +127,7 @@ sub authenticate_pam_user {
 
     # user need to be able to read /etc/passwd /etc/shadow
 
-    my $pamh = Authen::PAM->new('common-auth', $username, sub {
+    my $pamh = Authen::PAM->new('proxmox-mailgateway-auth', $username, sub {
 	my @res;
 	while(@_) {
 	    my $msg_type = shift;
-- 
2.39.2

[pmg-devel] [PATCH api 2/2] pam: set PAM_RHOST

[Wolfgang Bumiller]

This allows pam modules to restrict users by host. For
instance, you could restrict root@pam to only 127.0.0.1.

Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
---
 src/PMG/AccessControl.pm | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/PMG/AccessControl.pm b/src/PMG/AccessControl.pm
index 7926d4b..5ac4bc0 100644
--- a/src/PMG/AccessControl.pm
+++ b/src/PMG/AccessControl.pm
@@ -138,6 +138,12 @@ sub authenticate_pam_user {
 	return @res;
     });
 
+    if (my $rpcenv = PMG::RESTEnvironment->get()) {
+	if (my $ip = $rpcenv->get_client_ip()) {
+	    $pamh->pam_set_item(PAM_RHOST(), $ip);
+	}
+    }
+
     if (!ref($pamh)) {
 	my $err = $pamh->pam_strerror($pamh);
 	die "Error during PAM init: $err";
-- 
2.39.2

[pmg-devel] applied-series: [PATCH api 1/2] auth: set PAM context to 'proxmox-mailgateway-auth'

[Thomas Lamprecht]

Am 27/06/2023 um 15:39 schrieb Wolfgang Bumiller:
> This allows configuring PAM authentication for PMG via
> /etc/pam.d/proxmox-mailgateway-auth
> 
> Signed-off-by: Wolfgang Bumiller <w.bumiller@proxmox.com>
> ---
>  src/PMG/AccessControl.pm | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>

applied both patches, thanks!