Skip to content

feat: allow anonymous sessions for async status updates - #752

Merged
icewind1991 merged 2 commits into
mainfrom
feat/anonymous-session
Sep 16, 2026
Merged

icewind1991 merged 2 commits into
mainfrom
feat/anonymous-session

Conversation

@susnux

@susnux susnux commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

The biggest usecase would be e.g. chunked uploading, we should be able to early close the connection (prevent e.g. WAF issues) and then still give the UI asynchronous status updates. Also needed for other long running processes where the request itself might get terminated due to timeouts (network issues, WAF, etc).

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

The biggest usecase would be e.g. chunked uploading,
we should be able to early close the connection (prevent e.g. WAF
issues) and then still give the UI asynchronous status updates.
Also needed for other long running processes where the request itself
might get terminated due to timeouts (network issues, WAF, etc).

Assisted-by: ClaudeCode:claude-opus-5
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Comment thread src/user.rs Outdated
/// the user backend allows in user ids.
#[derive(Clone, Copy, Eq, PartialEq)]
enum IdKind {
User = 0,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

there is no need to assign values to the enum variants

Signed-off-by: Robin Appelman <robin@icewind.nl>

@susnux susnux left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ thanks for the pushed fix!

@icewind1991
icewind1991 merged commit ffb4430 into main Sep 16, 2026
42 of 43 checks passed
@icewind1991
icewind1991 deleted the feat/anonymous-session branch September 16, 2026 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants