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

Sync internal commits #184

Merged
merged 15 commits into from
Apr 5, 2024
Merged

Sync internal commits #184

merged 15 commits into from
Apr 5, 2024

Conversation

andrewhavck
Copy link
Contributor

No description provided.

pluveto and others added 15 commits March 25, 2024 11:50
@pluveto @drcaramelsyrup
tests(cache): add tests for purge method
15fb090 
Replicated-from: #18
Includes-commit: e5a3865
Includes-commit: f4af9ad
Includes-commit: bbfb1eb
Includes-commit: f74cf9f
@andrewhavck
Use escape_default() and limit the buffer size for InvalidHTTPHeader …
57d1228 
…errors when reading requests
@andrewhavck
Add example that routes to LB cluster based on request uri
8e67eab
@eaufavor
Allow response body to be modified.
39e8020
@hnakamur @andrewhavck
Add config for grace period and graceful shutdown timeout
728bbdc 
---
test: initialize ServerConf with added new fields


Replicated-from: #99
Includes-commit: f0dc59a
Includes-commit: c37e63b
@gumpt
bleeper: ignore Dockerfile
62c3892
@halzy @eaufavor
Fix voldemort types in public interface
cc3367a 
The `Fds` struct used in `ListenFds` in the `pingora::services::Service` trait
is not nameable. See https://rust-lang.github.io/rfcs/2145-type-privacy.html#~~~-voldemort-types-its-reachable-but-i-cant-name-it

This change makes both `ListenFds` and `Fds` public.

As a side-effect all of the methods on `Fds` are now public as well but are not documented.

Fixes: #145

Replicated-from: #146
Includes-commit: 6f835a3
@samurai00 @andrewhavck
Only compile pingora-openssl when openssl feature is enabled.
408f8b0 
Replicated-from: #131
Includes-commit: ae706cd
@jamesmunns @drcaramelsyrup
Expose currently pub(crate) configuration options, provide new Server…
d6563f5 
… constructor

---
Also expose the Version field

Includes-commit: acbc9e5
Includes-commit: d8fb7db
Replicated-from: #165
@drcaramelsyrup
Add doc comments for unstable fields in ServerConf
bad098b
@drcaramelsyrup
Add ServerConf::merge_with_opt for when Opt/ServerConf are used together
5d12e7b
@johnhurt
Add the doc configuration that enables the nice docs about detailing …
66b9124 
…which things are available on which features
@eaufavor @DimanNe
@cppcoffee @houseme @liloew @abruzzihraig
Fix typos and grammar issues in docs
383e6a9 
And other things.

Co-authored-by: DimanNe <[email protected]>
Co-authored-by: Xiaobo Liu <[email protected]>
Co-authored-by: houseme <[email protected]>
Co-authored-by: lilo <[email protected]>
Co-authored-by: Yang He <[email protected]>
@drcaramelsyrup
Clarify cache-control regex comments
beb575f 
Also update a few comment links for the newest HTTP RFCs.
@gdavidsson
Add/polish some comments
8c17dae
@andrewhavck andrewhavck merged commit 881db9e into main Apr 5, 2024
4 checks passed
@gideontong gideontong deleted the ahauck/apr-5-sync branch April 18, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gideontong gideontong gideontong approved these changes

@johnhurt johnhurt johnhurt approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.