Skip to content

Comments

chore: Forbid unsafe code with a global lint#348

Merged
gtema merged 1 commit intomainfrom
unsafe
Nov 10, 2025
Merged

chore: Forbid unsafe code with a global lint#348
gtema merged 1 commit intomainfrom
unsafe

Conversation

@gtema
Copy link
Collaborator

@gtema gtema commented Nov 10, 2025

Add a global lint prohibiting use of unsafe code. Luckily we have no
unsafe code anyway.

Fixes: #328

Add a global lint prohibiting use of unsafe code. Luckily we have no
unsafe code anyway.

Fixes: #328
@gtema gtema added this to Keystone Nov 10, 2025
@github-project-automation github-project-automation bot moved this to Backlog in Keystone Nov 10, 2025
@github-actions
Copy link

🐰 Bencher Report

Branchunsafe
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.19 µs
(+5.09%)Baseline: 1.13 µs
1.24 µs
(96.17%)
🐰 View full continuous benchmarking report in Bencher

@github-actions
Copy link

🦢 Load Test Results

Goose Attack Report

Plan Overview

Action Started Stopped Elapsed Users
Increasing 25-11-10 08:35:53 25-11-10 08:35:55 00:00:02 0 → 4
Maintaining 25-11-10 08:35:55 25-11-10 08:36:25 00:00:30 4
Decreasing 25-11-10 08:36:25 25-11-10 08:36:25 00:00:00 0 ← 4

Request Metrics

Method Name # Requests # Fails Average (ms) Min (ms) Max (ms) RPS Failures/s
GET 21651 0 5.04 4 9 721.70 0.00
Aggregated 21651 0 5.04 4 9 721.70 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 5 5 5 5 6 6 6 9
Aggregated 5 5 5 5 6 6 6 9

Status Code Metrics

Method Name Status Codes
GET 21,651 [200]
Aggregated 21,651 [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 21651 0 5.06 4 9 721.70 0.00
Aggregated 21651 0 5.04 4 9 721.70 0.00

Scenario Metrics

Transaction # Users # Times Run Average (ms) Min (ms) Max (ms) Scenarios/s Iterations
ListUsers 4 21647 5.06 4 9 721.57 5411.75
Aggregated 4 21647 5.06 4 9 721.57 5411.75

Error Metrics

Method Name # Error

View full report

@gtema gtema merged commit 264a528 into main Nov 10, 2025
25 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Keystone Nov 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Unsafe Code Analysis

1 participant