Skip to content

Add skip_headers argument#27

Open
ArtyomKomarenko wants to merge 1 commit intoofw:masterfrom
ArtyomKomarenko:skip-headers
Open

Add skip_headers argument#27
ArtyomKomarenko wants to merge 1 commit intoofw:masterfrom
ArtyomKomarenko:skip-headers

Conversation

@ArtyomKomarenko
Copy link

@ArtyomKomarenko ArtyomKomarenko commented Oct 13, 2021

By default returned string contains all headers including auto-generated ['accept', 'accept-encoding', 'connection', 'user-agent', 'content-length']. But these headers are not necessary to execute request by curl in terminal, and sometimes they can obstruct it execution. To avoid that case skip_headers=True can be passed to to_curl method.

@ofw
Copy link
Owner

ofw commented May 25, 2025

Hi! If you think that this change is still relevant please rebase it onto the main branch.

Also I'd suggest to provide the skip_headers param to the to_curl function for any users to be able to skip any list of headers not the hardcoded ones.

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