Skip to content

Conversation

@abhinavsingh
Copy link
Owner

  • [HttpParser] Memory view compliant
  • [New Flags] --max-sendbuf-size and --cache-requests
  • [Flag] Fix --min-compression-length usage
  • [Responses] okResponse now always adds a Content-Length header
  • [Requests] build_http_request now adds a Content-Length header when body is present
  • [Requests] build_http_request now adds a User-Agent header if one is not already provided and no_ua kwarg is not used
  • [Tutorial] Jupyter notebook based tutorial about proxy.py internals and developer API

abhinavsingh and others added 11 commits January 24, 2022 19:20
[Core] Invoke `selector.close` on shutdown
* Start of post encryption tests

* Assertion on post encryption callback

* Add `--cache-requests` flag

* Clean up `on_client_data` API as we no longer have a chain within core http protocol handler

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix flake8 warnings

* Fix `inconsistent-return-statements`

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Refactor into an internal task submodule of work

* As context managers

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Add missing license

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* `jupyter` notebook based tutorial

* Move within `tutorial` directory

* Fix spell

* Add `as_non_blocking` option during wrap

* `as_non_blocking=False` by defaut
* Add `--max-sendbuf-size` flag which now defaults to 64Kb

* Use `server_recvbuf_size` flag with base tunnel implementation

* isort

* Add to readme
* `min_compression_length` consistency, it was used as `min_compression_limit` at a few places

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* revert back web server route

* Move `serve_static_file` as a staticmethod within web plugin base

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* It hangs because of no content-length or connection close header

* Fix tests
* Add a response generation jupyter notebook

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Make codespell happy

* precommit codespell exclude

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Avoid using `tobytes` where possible

* `send` accepts `Union[memoryview, bytes]` now

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Remove usage of `tobytes`

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Fix chunk parser

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Lint fixes

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* Add plugin tests, responses notebook and enhancements to `build_http_packet`

* Add js code snip for ws example

* Fix tests

* ignore all ipynb from codespell

* ignore all ipynb from codespell

* Fix tests and doc spell
@abhinavsingh abhinavsingh added the bot:chronographer:skip PR using this label is exempted from CHANGELOG management label Jan 26, 2022
@codecov
Copy link

codecov bot commented Jan 26, 2022

Codecov Report

Attention: Patch coverage is 86.06061% with 46 lines in your changes missing coverage. Please review.

Project coverage is 88.04%. Comparing base (3858f3a) to head (3fd608e).
Report is 100 commits behind head on master.

Files Patch % Lines
proxy/core/work/task/remote.py 52.00% 12 Missing ⚠️
proxy/core/work/task/local.py 56.00% 11 Missing ⚠️
proxy/http/handler.py 50.00% 1 Missing and 2 partials ⚠️
proxy/common/utils.py 88.88% 1 Missing and 1 partial ⚠️
proxy/core/base/tcp_tunnel.py 0.00% 2 Missing ⚠️
proxy/core/work/task/task.py 60.00% 2 Missing ⚠️
proxy/http/server/plugin.py 86.66% 1 Missing and 1 partial ⚠️
proxy/http/server/web.py 60.00% 2 Missing ⚠️
proxy/plugin/cache/store/disk.py 66.66% 1 Missing and 1 partial ⚠️
proxy/plugin/modify_post_data.py 60.00% 1 Missing and 1 partial ⚠️
... and 6 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1068      +/-   ##
==========================================
+ Coverage   87.32%   88.04%   +0.72%     
==========================================
  Files         160      166       +6     
  Lines        6683     6861     +178     
  Branches      675      687      +12     
==========================================
+ Hits         5836     6041     +205     
+ Misses        741      707      -34     
- Partials      106      113       +7     
Flag Coverage Δ
pytest 87.87% <86.06%> (+0.72%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@abhinavsingh abhinavsingh merged commit 558a430 into master Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:skip PR using this label is exempted from CHANGELOG management

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants