refactor(server::client::auth): Approval flow for first-time connections and simplified to keep state on stack #26

Merged

Skipper merged 4 commits from push-xxmwpvvwnllx into main 2026-03-12 15:22:09 +00:00

Conversation 0 Commits 4 Files Changed 12 +590 -283

Skipper commented 2026-03-12 15:13:50 +00:00

Owner

Copy Link

No description provided.

Skipper added 40 commits 2026-03-12 15:13:51 +00:00

housekeeping(server): trimmed-down dependencies 0a8e1dce3f

security(server): cargo-vet proper init 9f33277a4f

tests(server): UserAgent invalid bootstrap token 1145642255

test(user-agent): add challenge-response auth flow test f769c9119b

ci: add server linting pipeline for Rust code quality checks 11f1caa6da

feat(server): boot mechanism 45acb45a05

feat(unseal): add unseal protocol support for user agents a55221573b

feat(server): UserAgent seal/unseal 281fbcb31d

security(server::key_holder): replaced nonce-caching with exclusive transaction fetching nonce from the database 4700bc407e

refactor(keyholder): rename KeyHolderActor to KeyHolder and optimize db connection lifetime bbaed3fb97

feat(server::key_holder): ability to remotely get current state 6c8a67c520

feat(server::key_holder): unique index on (root_key_id, nonce) to avoid nonce reuse 46a3c1768c

chore(supply-chain): update cargo-vet audits and trusted publishers 9ec465706a

refactor(actors): rename BootstrapActor to Bootstrapper b05cdeec66

fix(ci): add clippy installation in mise.toml bb58868333

fix(ci): add protoc installation for lints 08af101b2e

test(keyholder): remove unused imports from test modules 2ffd60973d

feat(server::user-agent): Unseal implemented 8f5d4cc385

feat(keyholder): add seal method and unseal integration tests c6e13dc476

refactor(server): actors reorganization & linter fixes 4a84fe9339

refactor(server): separated global actors into their own handle e118eceb85

refactor(server::tests): moved integration-like tests into tests/ d81120f59c

refactor(server): reogranized actors, context, and db modules into <dir>/mod.rs structure 925c7a211f

misc: create license and readme 4d1f047baf

feat(proto): add URL parsing and TLS certificate management 9dca7aff27

housekeeping(useragent): rename 7b57965952

refactor(transport): implemented Bi stream based abstraction for actor communication with next loop override b8afd94b21

refactor(transport): simplify converters f5a5c62181

feat(useragent): initial connection impl 6deec731e2

tests(user-agent): basic auth tests similar to server 86f8feb291

refactor(transport): convert Bi trait to use async_trait 657f47e32f

chore(server): update Cargo.lock dependencies 06f4d628db

refactor(server::{user_agent, client}): move auth part to separate function to not to pollute actor session with one-time concerns 54d0fe0505

refactor: consolidate auth messages into client and user_agent packages 4b4a8f4489

Merge pull request 'refactor(server::{user_agent, client}): move auth part to separate function to not to pollute actor session with one-time concerns' (#24) from push-upvpzwvlwyvs into main ... 4beb34764d

Reviewed-on: #24

feat(server): broker agent for inter-actor coordination cb05407bb6

feat(server::client): approval flow through user-agent on first-time client connects 168290040c

feat(server::client): proper connect error b3a67ffc00

feat(server::{router, useragent}): inter-actor approval coordination 606a1f3774

refactor(server::client::auth): removed state machine and added approval flow coordination ba86d18250

Skipper merged commit ec36e5c2ea into main 2026-03-12 15:22:09 +00:00

Skipper deleted branch push-xxmwpvvwnllx 2026-03-12 15:22:09 +00:00

Skipper referenced this issue from a commit 2026-03-12 15:22:11 +00:00

Merge pull request 'refactor(server::client::auth): Approval flow for first-time connections and simplified to keep state on stack' (#26) from push-xxmwpvvwnllx into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

Compat

Breaking

Breaking change that won't be backward compatible

Difficulty

High

3

Hard difficulty. Experience needed to fix: A lot.

Difficulty

Low

1

Easy difficulty. Experience needed to fix: Not much. Good first issue.

Difficulty

Medium

2

Medium difficulty. Experience needed to fix: Intermediate.

Kind

Bug

This is a bug.

Kind

Enhancement

Improve existing functionality.

Kind

Feature

New functionality

Kind

RFC

Request for comments

Kind

Security

This is security issue

Kind

Testing

This is testing issue.

Kind

Tracking issue

An issue tracking the progress of sth. like the implementation of an RFC

Priority

Critical

1

The priority is critical

Priority

High

2

The priority is high

Priority

Low

4

The priority is low

Priority

Medium

3

The priority is medium

Reviewed

Confirmed

1

Issue has been confirmed

Reviewed

Duplicate

2

This issue or pull request already exists

Reviewed

Invalid

3

Invalid issue

Reviewed

Pending

4

Requires review by dev

Reviewed

Won't Fix

3

This issue won't be fixed

Status

Abandoned

3

Somebody has started to work on this but abandoned work

Status

Blocked

1

Something is blocking this issue or pull request

Status

Need More Info

2

Feedback is required to reproduce issue or to continue work

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

CleverWild Skipper goodvin testuser

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MarketTakers/arbiter#26