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

feat: Experimental support for reqwest #839

Merged
merged 30 commits into from
Nov 26, 2020
Merged

feat: Experimental support for reqwest #839

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
311f4af
ref: Make forwarded endpoint requests go through upstream actor
untitaker Nov 16, 2020
042ce66
bundle a bunch of options into a struct for clippy
untitaker Nov 16, 2020
bd67a83
clean up some trait definitions
untitaker Nov 17, 2020
3158e11
Update CHANGELOG.md
untitaker Nov 17, 2020
edecbe6
abstract clientrequestbuilder away
untitaker Nov 17, 2020
be97db6
finish prep for adding reqwest
untitaker Nov 17, 2020
688d7c8
wip
untitaker Nov 17, 2020
6ea7c83
wip
untitaker Nov 18, 2020
87bbb69
split out http abstraction into own mod
untitaker Nov 18, 2020
402b40b
Merge remote-tracking branch 'origin/master' into ref/forward-through…
untitaker Nov 18, 2020
51c3f13
Merge branch 'ref/forward-through-upstream' of github.com:getsentry/r…
untitaker Nov 18, 2020
b7b6a8b
implement content-encoding and restrict http abstraction
untitaker Nov 18, 2020
d27b3ba
fix is_network_error
untitaker Nov 19, 2020
196c259
fix changelog
untitaker Nov 19, 2020
058eb97
test both http backends simultaneously
untitaker Nov 19, 2020
24c7251
fix broken "links"
untitaker Nov 19, 2020
56c7926
apply some of the feedback
untitaker Nov 23, 2020
d85610d
update todo
untitaker Nov 23, 2020
8536d0b
Merge remote-tracking branch 'origin/master' into ref/forward-through…
untitaker Nov 23, 2020
1af775c
fix some more of haproxy tests
untitaker Nov 23, 2020
424d909
refactor http error handling to use dedicated enum
untitaker Nov 23, 2020
b62e32b
implement limits for reqwest
untitaker Nov 23, 2020
d3b84e6
clean up http api a bit
untitaker Nov 23, 2020
161fee7
make gzip tests stronger
untitaker Nov 24, 2020
a4297ca
just consume response body for reqwest as well
untitaker Nov 24, 2020
6dbed35
Merge remote-tracking branch 'origin/master' into ref/forward-through…
untitaker Nov 25, 2020
5e6eb40
fix linters
untitaker Nov 25, 2020
c4b5570
streaming compressed body
untitaker Nov 25, 2020
6d9b303
add caveat
untitaker Nov 25, 2020
4bc6cc6
move function
untitaker Nov 26, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,8 @@

**Internal**:

- Add *experimental* support for picking up HTTP proxies from the regular environment variables. This feature needs to be enabled by setting `http: client: "reqwest"` in your `config.yml`. ([#839](https://github.com/getsentry/relay/pull/839))
untitaker marked this conversation as resolved.
Show resolved Hide resolved
- Refactor transparent request forwarding for unknown endpoints. Requests are now entirely buffered in memory and occupy the same queues and actors as other requests. This should not cause issues but may change behavior under load. ([#839](https://github.com/getsentry/relay/pull/839))
- Add reason codes to the `X-Sentry-Rate-Limits` header in store responses. This allows external Relays to emit outcomes with the proper reason codes. ([#850](https://github.com/getsentry/relay/pull/850))
- Emit metrics for outcomes in external relays. ([#851](https://github.com/getsentry/relay/pull/851))

Expand Down
101 changes: 59 additions & 42 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading