Skip to content

Conversation

bmurray
Copy link

@bmurray bmurray commented Mar 5, 2025

This adds support to v1, tested against a local Firecrawl server.

@bmurray
Copy link
Author

bmurray commented Mar 5, 2025

This should also resolve #7 as it allows the default HTTP client to be configured

It also adds cancellation of requests via a Context.

@MananDesai54
Copy link

MananDesai54 commented May 19, 2025

There is issue with this change, the JSON unmarshal for Search response is giving error cannot marshal array for description of type string. Can you suggest me what should be correct type? I'll make the change and raise the PR

@rafaelsideguide
Copy link
Collaborator

Hey @bmurray! Thanks a lot for your contribution to the repo—really appreciate it. I went ahead and added some tests and resolved the conflicts with main. Could you take a look when you have a moment?

And sorry for the delay in reviewing your PR. I’ll be keeping a closer eye on this repo moving forward.

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.

3 participants