From: Shannon Sterz <s.sterz@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox{, -backup} 0/4] http only cookie based tickets for pbs
Date: Thu, 10 Jul 2025 15:50:06 +0200 [thread overview]
Message-ID: <20250710135010.305861-1-s.sterz@proxmox.com> (raw)
# Summary
this series adds an authentication flow based on http only cookies for
proxmox backup server. at the moment this authentication flow is opt-in
in order to not break older clients that may still rely on the previous
authentication flow.
this series is split into three parts:
1. prepare proxmox-auth-api to be used with extjs and implements a new
ticket endpoint for pbs. this endpoint requires clients to provide a
boolean parameter `http-only` as `true` for it to switch to the new
http-only based authentication flow.
2. adapt proxmox backup server's ui components to always use the http
only based authentication flow. this should make cookies
inaccessible to any javascript-based attack in the browser,
providing an extra layer of security.
3. prepare pbs-client for potential servers that may no longer provide
the previous authentication flow. the point of already adding this
now, is to be prepare update-hesitant users for a future without the
old authentication flow. if the old authentication flow is dropped in
the future, more users will already have a client version that can
adapt to the new flow.
# Why not opt the `pbs-client` into the new flow?
the client is deliberatelly not opted into the new authentication flow
for the following reasons:
- http only cookies are only an effective security mechanism within a
browser context. the client simply does not benefit from this extra
layer of protection. the attacks http only cookies protect against,
simply don't exist here.
- opting the client in unconditionally isn't possible. older pbs servers
would complain about the additional api parameters provided by the
client. this means:
+ the client would either need to try the http-only flow and once
that fails, fall back to the older authentication flow.
+ or query (and possibly cache) the server version and check if the
version is new enough to support the new authentication flow.
both approaches are more error-prone and produce additional network
overhead than simply not opting the client into the new flow. causing
api errors may also produce a lot of false warnings when monitoring pbs.
so there are no benefits, but potential downsides. hence, the client
will not yet be opted into the new authentication flow.
proxmox:
Shannon Sterz (1):
auth-api: include meta information required by extjs in api endpoints
proxmox-auth-api/src/api/access.rs | 13 ++++++++++---
1 file changed, 10 insertions(+), 3 deletions(-)
proxmox-backup:
Shannon Sterz (3):
api: access: add opt-in http only ticket authentication flow
ui: opt into the new http-only ticket authentication flow
client: adapt pbs client to also handle http-only flows correctly
pbs-client/src/http_client.rs | 70 ++++++++++++++++++++++++++++---
src/api2/access/mod.rs | 77 +++++++++++++++++++++++++++++++++--
www/Application.js | 12 +++++-
www/LoginView.js | 4 +-
www/MainView.js | 1 +
www/Utils.js | 6 +++
6 files changed, 159 insertions(+), 11 deletions(-)
Summary over all repositories:
7 files changed, 169 insertions(+), 14 deletions(-)
--
Generated by git-murpp 0.8.1
_______________________________________________
pbs-devel mailing list
pbs-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pbs-devel
next reply other threads:[~2025-07-10 13:50 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-10 13:50 Shannon Sterz [this message]
2025-07-10 13:50 ` [pbs-devel] [PATCH proxmox 1/1] auth-api: include meta information required by extjs in api endpoints Shannon Sterz
2025-07-15 22:40 ` Thomas Lamprecht
[not found] ` <DBDBXGTI71WP.3V2J3DEMNK1DL@proxmox.com>
2025-07-22 20:21 ` Thomas Lamprecht
2025-07-23 15:18 ` Shannon Sterz
2025-07-10 13:50 ` [pbs-devel] [PATCH proxmox-backup 1/3] api: access: add opt-in http only ticket authentication flow Shannon Sterz
2025-07-23 12:57 ` Mira Limbeck
2025-07-23 13:58 ` Maximiliano Sandoval
2025-07-10 13:50 ` [pbs-devel] [PATCH proxmox-backup 2/3] ui: opt into the new http-only " Shannon Sterz
2025-07-10 13:50 ` [pbs-devel] [PATCH proxmox-backup 3/3] client: adapt pbs client to also handle http-only flows correctly Shannon Sterz
2025-07-23 12:56 ` [pbs-devel] [PATCH proxmox{, -backup} 0/4] http only cookie based tickets for pbs Mira Limbeck
2025-07-23 14:05 ` Maximiliano Sandoval
2025-07-23 15:15 ` Shannon Sterz
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=20250710135010.305861-1-s.sterz@proxmox.com \
--to=s.sterz@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.