From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: pdm-devel@lists.proxmox.com, Shannon Sterz <s.sterz@proxmox.com>
Subject: [pdm-devel] applied: [PATCH datacenter-manager] ui: avoid running tasks and remotes list loads when logged out
Date: Wed, 19 Nov 2025 21:39:38 +0100 [thread overview]
Message-ID: <176358477212.3307074.3754904082957998535.b4-ty@proxmox.com> (raw)
In-Reply-To: <20251114131305.210762-1-s.sterz@proxmox.com>
On Fri, 14 Nov 2025 14:13:05 +0100, Shannon Sterz wrote:
> the logic in the DatacenterManagerApp uses `Timeout`s that call
> `send_future` to continuously poll the remotes list and running tasks.
> the problem there is that given certain conditions these loads can
> happen over and over again, even if the user is logged out. this leads
> to 401 status codes being returned over and over again.
>
> the result is a race condition that can appear when a user logs in and
> one of those load requests is still in-flight. the user will log in
> showing the logged in ui. then one of those requests returns with a
> 401, which triggers an instant log out again. this is made more likely
> to happen by the fact that our api has a 3 second delay on requests
> that return a 401. this increases the likelihood of a load request
> still being in-flight while a login is happening.
>
> [...]
Applied, thanks!
[1/1] ui: avoid running tasks and remotes list loads when logged out
commit: 6a14f4571b0a8cf67949a13391671bba7c91c95b
_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel
prev parent reply other threads:[~2025-11-19 20:40 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-11-14 13:13 [pdm-devel] " Shannon Sterz
2025-11-19 20:39 ` Thomas Lamprecht [this message]
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=176358477212.3307074.3754904082957998535.b4-ty@proxmox.com \
--to=t.lamprecht@proxmox.com \
--cc=pdm-devel@lists.proxmox.com \
--cc=s.sterz@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.