~bigbes/tarantool-etcd · parade

main · last commit 1 month ago · 2npkec0r

← Back to the parade

tarantool-etcd-amf RoleGrantPermission accepts unsatisfiable ranges (empty key or key >= range_end) Lined Up

status: open P3 bug etcd-changelog-sweep
bd update tarantool-etcd-amf --claim bd close tarantool-etcd-amf
Created byEugene Blikh
Ownerbigbes@gmail.com
Created2026-07-17T13:20:52Z
Updated2026-07-17T13:20:52Z
Description
etcd's isValidPermissionRange returns ErrInvalidAuthMgmt for an empty key or key >= range_end (auth/store.go).

app/etcd/auth.lua:653-671 stores perm.key/range_end verbatim; the invalid permission is then silently stored and never matches anything — the admin believes a grant exists that cannot fire. Silent-misconfiguration class.

Everything else in this area is correct (nil-perm rejection, revoke semantics, bytes/base64 typing). errors.invalid_auth_management() already exists, so this is a small fix.
Acceptance criteria
Empty key or key >= range_end rejected at grant time with the etcd error; existing valid grants unaffected; test pins both invalid shapes.

Depends on

No outgoing dependencies.

Depended on by

Nothing depends on this issue.

No comments.

  • Eugene Blikh created the issue · 2026-07-17T16:20:51Z
  • Eugene Blikh added label etcd-changelog-sweep · 2026-07-17T16:20:51Z
Stored rows — what this pane was built from, as read
issues 1 row
id tarantool-etcd-amf
content_hash 291d4ae16a4b9432925311b49c11ba8000cce715301d29e6bdff95860fbdd176
title RoleGrantPermission accepts unsatisfiable ranges (empty key or key >= range_end)
description etcd's isValidPermissionRange returns ErrInvalidAuthMgmt for an empty key or key >= range_end (auth/store.go). app/etcd/auth.lua:653-671 stores perm.key/range_end verbatim; the invalid permission is then silently stored and never matches anything — the admin believes a grant exists that cannot fire. Silent-misconfiguration class. Everything else in this area is correct (nil-perm rejection, revoke semantics, bytes/base64 typing). errors.invalid_auth_management() already exists, so this is a small fix.
design
acceptance_criteria Empty key or key >= range_end rejected at grant time with the etcd error; existing valid grants unaffected; test pins both invalid shapes.
notes
status open
priority 3
issue_type bug
assignee NULL
estimated_minutes NULL
created_at 2026-07-17T13:20:52Z
created_by Eugene Blikh
owner bigbes@gmail.com
updated_at 2026-07-17T13:20:52Z
closed_at NULL
closed_by_session
external_ref NULL
spec_id
compaction_level 0
compacted_at NULL
compacted_at_commit NULL
original_size NULL
sender
ephemeral 0
wisp_type
pinned 0
is_template 0
mol_type
work_type
source_system
metadata <binary>
source_repo
close_reason
event_kind
actor
target
payload
await_type
await_id
timeout_ns 0
waiters
hook_bead
role_bead
agent_state
last_activity NULL
role_type
rig
due_at NULL
defer_until NULL
no_history 0
started_at NULL
is_blocked 0
labels 1 row
issue_id tarantool-etcd-amf
label etcd-changelog-sweep
events 2 rows
id 019f703c-aa8f-7d32-b302-3867fde8f1f8
issue_id tarantool-etcd-amf
event_type created
actor Eugene Blikh
old_value
new_value
comment NULL
created_at 2026-07-17T16:20:51Z
id 019f703c-aa90-788b-995a-d69e6033a7dd
issue_id tarantool-etcd-amf
event_type label_added
actor Eugene Blikh
old_value NULL
new_value NULL
comment Added label: etcd-changelog-sweep
created_at 2026-07-17T16:20:51Z