Skip to content

Fix duplicate 'status' key silently overridden in get_request_result response - #2480

Open
herniqeu wants to merge 1 commit into
Keeper-Security:masterfrom
herniqeu:fix/api-command-dup-status-key
Open

herniqeu wants to merge 1 commit into
Keeper-Security:masterfrom
herniqeu:fix/api-command-dup-status-key

Conversation

@herniqeu

@herniqeu herniqeu commented Nov 1, 2026

Copy link
Copy Markdown

Two status keys in the same dict literal: the later one wins, so the payload can never contain status="error" and the first entry is dead code. Behavior is unchanged by this patch; it only removes the shadowed key and keeps the winning value explicit. If a generic "error" was intended, drop the last line instead and I will update the PR.

This branch has not been deployed

No deployments
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