mirror of
https://github.com/mendableai/firecrawl.git
synced 2025-12-31 01:02:34 +00:00
21103b5a58
* fix: convert timeout from milliseconds to seconds in Python SDK - Fix timeout conversion in scrape_url method (line 596) - Fix timeout conversion in _post_request method (line 2207) - Add comprehensive tests for timeout functionality - Resolves issue #1848 The Python SDK was incorrectly passing timeout values in milliseconds directly to requests.post() which expects seconds, causing timeouts to be 1000x longer than intended (e.g. 60s became 16.6 hours). Co-Authored-By: rafael@sideguide.dev <rafael@sideguide.dev> * fix: handle timeout=0 edge case in conversion logic - Change condition from 'if timeout' to 'if timeout is not None' - Ensures timeout=0 is converted to 5.0 seconds instead of None - All timeout conversion tests now pass (5/5) Co-Authored-By: rafael@sideguide.dev <rafael@sideguide.dev> * feat: change default timeout from None to 30s (30000ms) - Update all timeout parameter defaults from None to 30000ms across SDK - ScrapeOptions, MapParams, and all method signatures now default to 30s - Update tests to verify new default timeout behavior (35s total with 5s buffer) - Add test for _post_request when no timeout key is present in data - Maintains backward compatibility for explicit timeout values - All 6 timeout conversion tests pass Co-Authored-By: rafael@sideguide.dev <rafael@sideguide.dev> --------- Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Co-authored-by: rafael@sideguide.dev <rafael@sideguide.dev>