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.

Sign up for GitHub

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

Jump to bottom

Feature: Tool to convert cURL to Escambo. Issue #33 #74

Open
wants to merge 37 commits into
base: main
Choose a base branch
from
Open

Feature: Tool to convert cURL to Escambo. Issue #33 #74

wants to merge 37 commits into from

Conversation

lindolo25
Copy link

Hi @CleoMenezesJr, This is a basic implementation of the cURL import tool. It should work for must common cases but I'm expecting it will not be a complete solution just jet. I will continue to support it as issues or requests arise.

The window file required big changes to try to re-use as much as possible the functionality between the app initialization, UI functionality and the import.

Please review the PR and let me know your thoughts. I hope this helps and makes it to a release soon.

Resolve #33

lindolo25 and others added 30 commits July 2, 2023 23:29
@lindolo25
🔧 Create cURL parser class
c88e025
@lindolo25
🔧 Create import dialog, class and template
1df6550
@lindolo25
🔧 Add new resources to meson build and gresource
4388ad4
@lindolo25
🔧 Add import main action
52b8689
@lindolo25
🪲 Fix meson build resource typo
f78dcca
@lindolo25
🔧 Add action to populate URL from import
bad02a5
@lindolo25
🔧 Include 'method' in populate action from import
6d7b108
@lindolo25
🪲 Fix error raised by an invalid cURL format.
43428eb
@lindolo25
🔧 Add type definitions for properties
f7ea64b
@lindolo25
🔧 Handle parse exception and abort import
225b25e
@lindolo25
🔧 Improve authorization getter property.
1d9a016
@lindolo25
🔧 Implement curl population of authorization page
f851d6d
@lindolo25
🔧 Add reomve callback to PopulatorEntry constructor.
a9b7e00
@lindolo25
🔧 Split entries creation and update into reusable peices of functiona…
d746883 
…lity. file reading, add item to file, create PopulatorEntry
@lindolo25
🔧 populate headers from cURL import.
21c305c
@lindolo25
🔧 Implement prepopulation of header switch from cURL import.
62ffe87
@lindolo25
🔧 add support for cookies to the CurlParser. Changes to the arguments…
adfaa1e 
… configuration.
@lindolo25
🔧 Add cookies property to CurlParser.
927f98e
@lindolo25
🔧 implement form population from curl for cookies.
b1f5dfb
@lindolo25
🔧 Improve cookies population by detecting other properties like Domain
246cd49
@lindolo25
🔧 Unify cookies section in update_states()
ec7845e
@lindolo25
🐛 Fix labels for params in PopulatorEntry
fd23a2e
@lindolo25
🔧 Refactor code to allow re-use for params population from curl
a4812e1
@lindolo25
🔧 Implement params for CurlParser
727d746
@lindolo25
🔧 Add params to populate_from_curl()
7d3fa94
@lindolo25
🐛 Fix url property to only return path without params.
529be98
@lindolo25
🔥 Remove print() statements in window
9df05a0
@lindolo25
🔧 Update population of headers, cookies and authorization at start.
3635ce1
@lindolo25
♻️ Update types and functions rename.
94b4dd9
@lindolo25
🔧 Update to set edit button as hidden for params in PopulatorEntry
8a12e40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature: Tool to convert cURL to Escambo
1 participant
@lindolo25