Skip to content

fix: harden tunnels and code quality checks - #173

Merged
benfavre merged 1 commit into
mainfrom
fix/code-quality-audit
Sep 30, 2026
Merged

benfavre merged 1 commit into
mainfrom
fix/code-quality-audit

Conversation

@pedrokarim

Copy link
Copy Markdown
Collaborator

Résumé

  • les tunnels local et SOCKS ouvrent leur port avant de rendre la main : Ok garantit désormais une écoute effective, et un port déjà occupé renvoie PortInUse immédiatement
  • les tests réseau se connectent sans boucle d’attente et réservent des ports éphémères uniques, ce qui supprime leur instabilité en parallèle
  • corrige les deux alertes de cargo clippy --all-targets dans les tests de platform_attention et platform_review
  • l’accueil User trie les sites une seule fois par rendu et ne clone plus l’annuaire ni la liste des demandes
  • deny.toml contrôle les dépendances non maintenues sur tout le graphe

Vérification

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets --no-deps -- -D warnings
  • cargo test --workspace
  • cargo deny check advisories
  • cargo audit

- Bind local and SOCKS forward listeners before start returns, so Ok
  means the port is accepting connections and a pre-bound port reports
  PortInUse synchronously instead of failing inside the spawned task.
- Make tunnel tests connect immediately and claim unique ephemeral
  ports, removing parallel flakiness.
- Fix clippy --all-targets findings in platform attention and review
  worktree tests.
- Partition User home sites once per render with cached sort keys and
  borrow the site directory and request list instead of cloning them.
- Check unmaintained RustSec advisories across the whole dependency
  graph in deny.toml.
@benfavre
benfavre merged commit 2731fc3 into main Sep 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants