Skip to content

update(idemix): hardening - #2136

Open
adecaro wants to merge 1 commit into
mainfrom
idemix-hardening
Open

update(idemix): hardening#2136
adecaro wants to merge 1 commit into
mainfrom
idemix-hardening

Conversation

@adecaro

@adecaro adecaro commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

This PR introduces idemix hardening.

@adecaro adecaro added this to the Q3/26 milestone Aug 4, 2026
@adecaro adecaro self-assigned this Aug 4, 2026
@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

📊 Token Validation Benchmark

Comparison of this PR against the base branch. 🟢 improvement · 🔴 regression · ➖ within ±1.0% noise.

Variant Benchmark Params Workers TPS (base → PR) Δ TPS
csp BenchmarkAPIGRPC f=1, nc=4, w=token-validation-service 4 106 → 96 🔴 -8.7%
csp BenchmarkLocalTokenValidation out-tokens=2in-tokens=2 4 135 → 123 🔴 -9.3%
ipa BenchmarkAPIGRPC f=1, nc=4, w=token-validation-service 4 78 → 72 🔴 -6.7%
ipa BenchmarkLocalTokenValidation out-tokens=2in-tokens=2 4 68 → 64 🔴 -6.5%

@AkramBitar AkramBitar left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Angelo De Caro <adc@zurich.ibm.com>
@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown

📊 Token Validation Benchmark

Comparison of this PR against the base branch. 🟢 improvement · 🔴 regression · ➖ within ±1.0% noise.

Variant Benchmark Params Workers TPS (base → PR) Δ TPS
csp BenchmarkAPIGRPC f=1, nc=4, w=token-validation-service 4 118 → 107 🔴 -9.2%
csp BenchmarkLocalTokenValidation out-tokens=2in-tokens=2 4 115 → 104 🔴 -10.0%
ipa BenchmarkAPIGRPC f=1, nc=4, w=token-validation-service 4 90 → 85 🔴 -6.2%
ipa BenchmarkLocalTokenValidation out-tokens=2in-tokens=2 4 76 → 71 🔴 -7.0%

@adecaro

adecaro commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

waiting for IBM/idemix#89 to get merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants