Skip to content
New issue

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

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

Already on GitHub? # to your account

Reverse proxy plugin constructor now accepts client connection object #1235

Merged
merged 1 commit into from
Jul 30, 2022

Conversation

abhinavsingh
Copy link
Owner

@abhinavsingh abhinavsingh added the bot:chronographer:skip PR using this label is exempted from CHANGELOG management label Jul 30, 2022
@codecov
Copy link

codecov bot commented Jul 30, 2022

Codecov Report

Merging #1235 (0af634f) into develop (dc06ea4) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop    #1235      +/-   ##
===========================================
+ Coverage    87.16%   87.17%   +0.01%     
===========================================
  Files          166      166              
  Lines         7019     7025       +6     
  Branches       721      721              
===========================================
+ Hits          6118     6124       +6     
  Misses         779      779              
  Partials       122      122              
Flag Coverage Δ
pytest 86.97% <100.00%> (+0.01%) ⬆️

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

Impacted Files Coverage Δ
proxy/http/server/plugin.py 92.18% <100.00%> (+0.80%) ⬆️
proxy/http/server/reverse.py 82.81% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@abhinavsingh abhinavsingh merged commit 9d9bbd1 into develop Jul 30, 2022
@abhinavsingh abhinavsingh deleted the reverse-proxy-client branch July 30, 2022 13:33
# for free to join this conversation on GitHub. Already have an account? # 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.

[Reverse Proxy] Client IP Address
1 participant