all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Christian Ebner <c.ebner@proxmox.com>
To: Maximiliano Sandoval <m.sandoval@proxmox.com>,
	pbs-devel@lists.proxmox.com
Subject: Re: [PATCH backup v2 2/3] docs: client: document further password constrains
Date: Mon, 23 Feb 2026 08:58:55 +0100	[thread overview]
Message-ID: <3c5eac3f-819c-4e3b-b258-22ae5e2f1f0f@proxmox.com> (raw)
In-Reply-To: <20260220123818.365489-3-m.sandoval@proxmox.com>

On 2/20/26 1:38 PM, Maximiliano Sandoval wrote:
> We leave the explicit newlines as "control character" might not mean
> much anything to some readers.

This is grammatically not correct, you might drop either the "much" or 
the "anything"

> 
> Signed-off-by: Maximiliano Sandoval <m.sandoval@proxmox.com>
> ---
>   docs/backup-client.rst | 7 ++++---
>   1 file changed, 4 insertions(+), 3 deletions(-)
> 
> diff --git a/docs/backup-client.rst b/docs/backup-client.rst
> index 40962f0e2..03a383d9c 100644
> --- a/docs/backup-client.rst
> +++ b/docs/backup-client.rst
> @@ -104,9 +104,10 @@ Environment Variables
>      wireguard, instead of using an HTTP proxy.
>   
>   
> -.. Note:: Passwords must be valid UTF-8 and may not contain newlines. For your
> -   convenience, Proxmox Backup Server only uses the first line as password, so
> -   you can add arbitrary comments after the first newline.
> +.. Note:: Passwords must be valid UTF-8 and may not contain newlines or any
> +   control characters in general. For your convenience, Proxmox Backup Server

nit: "... any other control characters." is a bit more concise.

> +   only uses the first line as password, so you can add arbitrary comments after
> +   the first newline.
>   
>   
>   System and Service Credentials





  reply	other threads:[~2026-02-23  7:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-20 12:38 [PATCH backup v2 0/3] fix #7054: client: remove trailing newlines from credentials Maximiliano Sandoval
2026-02-20 12:38 ` [PATCH backup v2 1/3] " Maximiliano Sandoval
2026-02-23  7:55   ` Christian Ebner
2026-02-23  9:33     ` Maximiliano Sandoval
2026-02-20 12:38 ` [PATCH backup v2 2/3] docs: client: document further password constrains Maximiliano Sandoval
2026-02-23  7:58   ` Christian Ebner [this message]
2026-02-20 12:38 ` [PATCH backup v2 3/3] client: rename password to blob Maximiliano Sandoval
2026-02-23 10:07 ` superseded: [PATCH backup v2 0/3] fix #7054: client: remove trailing newlines from credentials Maximiliano Sandoval

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3c5eac3f-819c-4e3b-b258-22ae5e2f1f0f@proxmox.com \
    --to=c.ebner@proxmox.com \
    --cc=m.sandoval@proxmox.com \
    --cc=pbs-devel@lists.proxmox.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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