feat: add configurable timeout for decompilation#123
Open
jethac wants to merge 2 commits intoLaurieWired:mainfrom
Open
feat: add configurable timeout for decompilation#123jethac wants to merge 2 commits intoLaurieWired:mainfrom
jethac wants to merge 2 commits intoLaurieWired:mainfrom
Conversation
- Replace requests with httpx for better connection pooling - Add tenacity for automatic retry on transient failures - Retry up to 3 times with exponential backoff on ConnectError/ConnectTimeout - Add connection-pooled HTTP client singleton
- Add timeout parameter to decompile_function_by_address (default: 120s, max: 600s) - Add timeout parameter to safe_get for per-request timeout control - Large/complex functions can now use longer timeouts to complete decompilation
bethington
approved these changes
Feb 11, 2026
bethington
left a comment
bethington
left a comment
There was a problem hiding this comment.
LGTM! This addresses the timeout issues users have been reporting (Issue #79). Clean implementation.
This was referenced Feb 11, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
tl,dr
Add configurable timeout parameter to
decompile_function_by_addressfor handling large/complex functions.Background
Large functions can take minutes to decompile. The fixed 30s timeout causes these to fail with timeout errors, leaving users unable to analyze complex code. Users need control over how long to wait for decompilation.
Changes
timeoutparameter todecompile_function_by_address(default: 120s, max: 1800s / 30min)timeoutparameter to internalsafe_get()for per-request timeout controlTesting
timeout=300, successfully completedtimeout=9999becomes 1800s)