GNU bug report logs - #62699
[PATCH] services: add pam-mount-volume-service-type

Previous Next

Full log

View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Ludovic Courtès <ludo <at> gnu.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#62699: closed ([PATCH] services: add pam-mount-volume-service-type)
Date: Wed, 09 Aug 2023 10:31:02 +0000

[Message part 1 (text/plain, inline)]

Your message dated Wed, 09 Aug 2023 12:30:23 +0200
with message-id <87fs4spkxs.fsf <at> gnu.org>
and subject line Re: bug#62699: [PATCH v2] services: add pam-mount-volume-service-type
has caused the debbugs.gnu.org bug report #62699,
regarding [PATCH] services: add pam-mount-volume-service-type
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
62699: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62699
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems

[Message part 2 (message/rfc822, inline)]

From: Brian Cully <bjc <at> spork.org>
To: guix-patches <at> gnu.org
Subject: [PATCH] services: add pam-mount-volume-service-type
Date: Thu, 06 Apr 2023 12:43:55 -0400

This patch allows adding additional volumes to be mounted at login 
time via PAM by way of ‘pam-mount-volume-service-type’. As an 
example usage, I use it to mount a CIFS share which requires 
authentication automatically on login without having to type my 
password twice (since my local system has the same username and 
password as the system hosting my CIFS share).

-bjc

[Message part 3 (message/rfc822, inline)]

From: Ludovic Courtès <ludo <at> gnu.org>
To: Brian Cully <bjc <at> spork.org>
Cc: glv <at> posteo.net, 62699-done <at> debbugs.gnu.org
Subject: Re: bug#62699: [PATCH v2] services: add pam-mount-volume-service-type
Date: Wed, 09 Aug 2023 12:30:23 +0200

[Message part 4 (text/plain, inline)]

Hello,

Brian Cully <bjc <at> spork.org> skribis:

> The `pam-mount-volumes-service-type' adds additional volumes to the
> pam-mount-service-type in addition to any that are already specified in
> `pam-mount-rules'.
>
> * doc/guix.texi (PAM Mount Volume Service): add documentation for
> `pam-mount-service-type'.
> * gnu/services/pam-mount.scm: new file.
> * Makefile.am: add pam-mount tests
> * tests/services/pam-mount.scm: new tests

Applied with the changes below.

Thanks!

Ludo’.

[Message part 5 (text/x-patch, inline)]

diff --git a/gnu/services/pam-mount.scm b/gnu/services/pam-mount.scm
index 8a38d6b1cc..dbb9d0285f 100644
--- a/gnu/services/pam-mount.scm
+++ b/gnu/services/pam-mount.scm
@@ -1,5 +1,6 @@
 ;;; GNU Guix --- Functional package management for GNU
 ;;; Copyright © 2019 Guillaume Le Vaillant <glv <at> posteo.net>
+;;; Copyright © 2023 Brian Cully <bjc <at> spork.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -220,4 +221,6 @@ (define pam-mount-volume-service-type
                 (extend append)
                 (default-value '())
                 (description
-                 "Volumes to be mounted during PAM-assisted login.")))
+                 "Mount remote volumes such as CIFS shares @i{via}
+@acronym{PAM, Pluggable Authentication Modules} when logging in, using login
+credentials.")))

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.