rasterstate/fj-demo
1
0
Fork 0
Code Issues 4 Pull requests Projects Releases 1 Packages Wiki Activity Actions

[*] Reject base62 codes longer than 11 chars (#2) #7

Closed
stephen wants to merge 1 commit from fix-long-codes into main
pull from: fix-long-codes
merge into: rasterstate:main
Conversation 1 Commits 1 Files changed 1 +4
stephen commented 2026-06-05 16:47:39 +00:00
Owner
Copy link

Closes #2. A 12-char code can overflow u64 mid-multiply; bound the length up front and return None instead.

Closes #2. A 12-char code can overflow u64 mid-multiply; bound the length up front and return None instead.
Reject base62 codes longer than 11 chars
All checks were successful
ci / test (pull_request) Successful in 21s
Details
c795ab29be
stephen reviewed 2026-06-05 16:48:45 +00:00
stephen left a comment
Copy link

One nit: worth a test at the 12-char boundary.

One nit: worth a test at the 12-char boundary.
stephen changed title from Reject base62 codes longer than 11 chars (#2) to [*] Reject base62 codes longer than 11 chars (#2) 2026-06-08 01:06:59 +00:00
stephen closed this pull request 2026-06-11 15:26:58 +00:00
All checks were successful
ci / test (pull_request) Successful in 21s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backlog
Accepted but not scheduled

  
blocked
Blocked or stalled waiting on external progress

  
converted
Converted into tracked work

  
opportunity
Potential product or business opportunity

  
p1
Priority 1

  
p2
Priority 2

  
p3
Priority 3

  
parked
Paused intentionally until later

No labels
backlog
blocked
converted
opportunity
p1
p2
p3
parked
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rasterstate/fj-demo!7
No description provided.