Skip to content

add benchmark tooling + docs + CI integration#82

Draft
GlenDC wants to merge 60 commits intomainfrom
feat/netbench
Draft

add benchmark tooling + docs + CI integration#82
GlenDC wants to merge 60 commits intomainfrom
feat/netbench

Conversation

@GlenDC
Copy link
Collaborator

@GlenDC GlenDC commented Jan 29, 2026

Ready for feedback on the initial version of benchmarking support for proxy code.

CI output example: #82 (comment)

Summary by Aikido

⚠️ Security Issues: 2 🔍 Quality Issues: 13 Resolved Issues: 0

🚀 New Features

  • Added blocked-event notifier and injected x-blocked-by header on blocks.

⚡ Enhancements

  • Introduced configurable WebClientConfig and transport module for web clients.
  • Added telemetry and env utilities including TelemetryConfig and server_identifier.

🔧 Refactors

  • Replaced DomainTrie with DomainMatcher across firewall rules and logic.
  • Refactored proxy server into ProxyServer builder and extracted HTTP creator.

More info

if you do use unwrap/expect you need to allow it
and hopefully with a good reason
@GlenDC
Copy link
Collaborator Author

GlenDC commented Feb 1, 2026

Ready for review @bitterpanda63 .
We can go through it together if you have questions.

It's a bit of a big one. I usually try to prevent these.
But I think it's warranted here.

version = "0.1.0"
edition = "2024"
name = "safechain-proxy-lib"
description = "Core library for safechain-proxy (cli)"
Copy link
Member

Choose a reason for hiding this comment

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

why cli here?


pub type EntryFormatter = Arc<dyn MalwareListEntryFormatter>;

pub const MALWARE_LIST_URI_STR_VSCODE: &str = "https://malware-list.aikido.dev/malware_vscode.json";
Copy link
Member

Choose a reason for hiding this comment

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

revert

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

instead of reverting I will just make small Prs for all these things individually. Afterwards I will simply get it merged into this branch again, from main.

@GlenDC GlenDC marked this pull request as draft February 4, 2026 17:54
@GlenDC
Copy link
Collaborator Author

GlenDC commented Feb 4, 2026

Demoted to draft again until all parts out of scope are merged into main via separate PRs or removed

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