Skip to content

feat: Enforce context for DB operations in identity#339

Merged
gtema merged 1 commit intomainfrom
db_context
Nov 4, 2025
Merged

feat: Enforce context for DB operations in identity#339
gtema merged 1 commit intomainfrom
db_context

Conversation

@gtema
Copy link
Collaborator

@gtema gtema commented Nov 4, 2025

Remove From trait implementation for the DB operations in the identity
provider to enforce using the helper method with specifying the context
information. This dramatically improves the error finding since the
error contain the reference to the operation that caused it.

Remove From trait implementation for the DB operations in the identity
provider to enforce using the helper method with specifying the context
information. This dramatically improves the error finding since the
error contain the reference to the operation that caused it.
@github-actions
Copy link

github-actions bot commented Nov 4, 2025

🐰 Bencher Report

Branchdb_context
Testbedubuntu-latest
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
microseconds (µs)
(Result Δ%)
Upper Boundary
microseconds (µs)
(Limit %)
fernet token/project📈 view plot
🚷 view threshold
1.16 µs
(+3.81%)Baseline: 1.12 µs
1.23 µs
(94.67%)
🐰 View full continuous benchmarking report in Bencher

@github-actions
Copy link

github-actions bot commented Nov 4, 2025

🦢 Load Test Results

Goose Attack Report

Plan Overview

Action Started Stopped Elapsed Users
Increasing 25-11-04 17:32:35 25-11-04 17:32:37 00:00:02 0 → 4
Maintaining 25-11-04 17:32:37 25-11-04 17:33:07 00:00:30 4
Decreasing 25-11-04 17:33:07 25-11-04 17:33:07 00:00:00 0 ← 4

Request Metrics

Method Name # Requests # Fails Average (ms) Min (ms) Max (ms) RPS Failures/s
GET 15848 0 7.06 5 14 528.27 0.00
Aggregated 15848 0 7.06 5 14 528.27 0.00

Response Time Metrics

Method Name 50%ile (ms) 60%ile (ms) 70%ile (ms) 80%ile (ms) 90%ile (ms) 95%ile (ms) 99%ile (ms) 100%ile (ms)
GET 7 7 7 7 8 8 9 14
Aggregated 7 7 7 7 8 8 9 14

Status Code Metrics

Method Name Status Codes
GET 15,848 [200]
Aggregated 15,848 [200]

Transaction Metrics

Transaction # Times Run # Fails Average (ms) Min (ms) Max (ms) RPS Failures/s
ListUsers
0.0 0 0 0.00 0 0 0.00 0.00
0.1 15848 0 7.09 5 14 528.27 0.00
Aggregated 15848 0 7.06 5 14 528.27 0.00

Scenario Metrics

Transaction # Users # Times Run Average (ms) Min (ms) Max (ms) Scenarios/s Iterations
ListUsers 4 15844 7.09 5 14 528.13 3961.00
Aggregated 4 15844 7.09 5 14 528.13 3961.00

Error Metrics

Method Name # Error

View full report

@gtema gtema merged commit 8a810f1 into main Nov 4, 2025
19 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.

1 participant

Comments