all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* [pve-devel] [PATCH docs] pveum: document correct path for ldap/realm credentials
@ 2025-03-28 12:07 Christoph Heiss
  2025-04-04 10:19 ` [pve-devel] applied: " Thomas Lamprecht
  0 siblings, 1 reply; 2+ messages in thread
From: Christoph Heiss @ 2025-03-28 12:07 UTC (permalink / raw)
  To: pve-devel

/etc/pve/priv/realm has been the default path since commit

  782b702db ("auth ldap/ad: make password a parameter for the api")

That commit was part of pve-access-control 6.0-7.

Replace /etc/pve/priv/ldap with /etc/pve/priv/realm, to avoid pointing
users to the deprecated path.

It's not used anywhere in our code too, except as (very) old fallback.

Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>
---
 pveum.adoc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/pveum.adoc b/pveum.adoc
index 0cc267e..c5fbd46 100644
--- a/pveum.adoc
+++ b/pveum.adoc
@@ -224,8 +224,8 @@ attribute would be `uid`.
 If {pve} needs to authenticate (bind) to the LDAP server before being
 able to query and authenticate users, a bind domain name can be
 configured via the `bind_dn` property in `/etc/pve/domains.cfg`. Its
-password then has to be stored in `/etc/pve/priv/ldap/<realmname>.pw`
-(for example, `/etc/pve/priv/ldap/my-ldap.pw`). This file should contain a
+password then has to be stored in `/etc/pve/priv/realm/<realmname>.pw`
+(for example, `/etc/pve/priv/realm/my-ldap.pw`). This file should contain a
 single line with the raw password.
 
 To verify certificates, you need to set `capath`. You can set it either
-- 
2.48.1



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [pve-devel] applied: [PATCH docs] pveum: document correct path for ldap/realm credentials
  2025-03-28 12:07 [pve-devel] [PATCH docs] pveum: document correct path for ldap/realm credentials Christoph Heiss
@ 2025-04-04 10:19 ` Thomas Lamprecht
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Lamprecht @ 2025-04-04 10:19 UTC (permalink / raw)
  To: Proxmox VE development discussion, Christoph Heiss

Am 28.03.25 um 13:07 schrieb Christoph Heiss:
> /etc/pve/priv/realm has been the default path since commit
> 
>   782b702db ("auth ldap/ad: make password a parameter for the api")
> 
> That commit was part of pve-access-control 6.0-7.
> 
> Replace /etc/pve/priv/ldap with /etc/pve/priv/realm, to avoid pointing
> users to the deprecated path.
> 
> It's not used anywhere in our code too, except as (very) old fallback.
> 
> Signed-off-by: Christoph Heiss <c.heiss@proxmox.com>
> ---
>  pveum.adoc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
>

applied, thanks!


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-04-04 10:19 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-03-28 12:07 [pve-devel] [PATCH docs] pveum: document correct path for ldap/realm credentials Christoph Heiss
2025-04-04 10:19 ` [pve-devel] applied: " Thomas Lamprecht

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal