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.
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
ref: Upgrade to Actix 1 #106
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm approving this because the code looks fine to me, ofc the tests need to be fixed
Tests are green now 🤷♂ @untitaker See 0b0cc2a. This commit contains the fix for failing CAB extraction. The doc comments describe the reason for this fix. Summary: The file is only namable, until |
FWIW, test failures were caused by file descriptor limits. With the logs at hand this was not completely obvious, but we can work on surfacing them better. |
This refactors the entire web layer including middlewares and restructures services. To speed up compilation, many
impl Trait
have been replaced with Boxes. Also, the library and binary crate have been merged into one. Rust tests also need less hacks now.The inner works of services (e.g. symbol lookup) has not been touched!
Since the diff is pretty unusable, I suggest to review the outcome directly. Ideally, start from
main
, follow toserver::run
andService::create
, and then look at the endpoints and middlewares.The Sentry middleware and utilities can later be moved into a new version of
sentry-actix
, and I would also propose to create a new crate,sentry-futures
.