Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add curl import functionality #123

Merged
merged 21 commits into from
Nov 10, 2024
Merged

Conversation

MikeBellika
Copy link
Contributor

This PR adds support for pasting a cURL command in the URL bar as discussed here: #73

Here's a quick demo:

Screen.Recording.2024-11-04.at.21.51.30.mov

In this demo you can see that the headers are parsed correctly, the data is parsed correctly and the --insecure flag disables the Verify SSL certificates option.

There might be some edge cases that it can't handle, in which case it will display a message to the user. Testing with curl requests from chrome seems to work.

@darrenburns darrenburns self-requested a review November 10, 2024 17:52
@darrenburns
Copy link
Owner

Thank you very much!

I've made some changes on top of your work to re-use some of the existing code, and added some extra type hinting. I'm going to merge this soon, and then work on some docs, and a few other small changes, then release.

Hopefully this will be live within the next week or so.

@darrenburns darrenburns merged commit c5e7e47 into darrenburns:main Nov 10, 2024
1 check passed
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.

2 participants