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

Overflow when using MergedObject #1516

Open
Sytten opened this issue Apr 28, 2024 · 0 comments
Open

Overflow when using MergedObject #1516

Sytten opened this issue Apr 28, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@Sytten
Copy link

Sytten commented Apr 28, 2024 •
edited

Expected Behavior

I upgraded from v5 to v7.0.2, it should work still.

Actual Behavior

overflow evaluating the requirement `http::Request<reqwest::async_impl::body::ImplStream>: std::marker::Send`
consider increasing the recursion limit by adding a `#![recursion_limit = "256"]` attribute to your crate (`proxy_api`)
required because it appears within the type `(Request<ImplStream>, Callback<Request<ImplStream>, Response<Body>>)`
required for `Chan<Envelope<Request<ImplStream>, Response<Body>>, Semaphore>` to implement `std::marker::Sync`
the full type name has been written to '/Users/sytten/Projects/Caido/proxy-backend/target/debug/deps/proxy_api-c28335827c5bb19d.long-type-8384231238600946750.txt'
required for `Arc<Chan<Envelope<Request<ImplStream>, Response<Body>>, Semaphore>>` to implement `std::marker::Send`
the full type name has been written to '/Users/sytten/Projects/Caido/proxy-backend/target/debug/deps/proxy_api-c28335827c5bb19d.long-type-17041696055801546513.txt'
required for `std::ptr::Unique<hyper::client::pool::Idle<hyper::client::client::PoolClient<reqwest::async_impl::body::ImplStream>>>` to implement `std::marker::Send`
required because it appears within the type `((Scheme, Authority), Vec<Idle<PoolClient<ImplStream>>>)`
required for `RawTable<((Scheme, Authority), Vec<Idle<PoolClient<ImplStream>>>)>` to implement `std::marker::Send`
the full type name has been written to '/Users/sytten/Projects/Caido/proxy-backend/target/debug/deps/proxy_api-c28335827c5bb19d.long-type-8221124596057829378.txt'
required for `std::sync::Mutex<hyper::client::pool::PoolInner<hyper::client::client::PoolClient<reqwest::async_impl::body::ImplStream>>>` to implement `std::marker::Sync`
1 redundant requirement hidden
required for `std::sync::Arc<std::sync::Mutex<hyper::client::pool::PoolInner<hyper::client::client::PoolClient<reqwest::async_impl::body::ImplStream>>>>` to implement `std::marker::Sync`
required for `std::sync::Arc<reqwest::async_impl::client::ClientRef>` to implement `std::marker::Send`
required because it captures the following types: `reqwest::async_impl::client::Client`, `std::ops::ControlFlow<std::result::Result<std::convert::Infallible, reqwest::error::Error>, reqwest::async_impl::client::Client>`, `impl futures::Future<Output = std::result::Result<reqwest::async_impl::response::Response, reqwest::error::Error>>`
required because it captures the following types: `&str`, `&std::path::Path`, `impl futures::Future<Output = std::result::Result<tokio::fs::File, std::io::Error>>`, `tokio::io::BufWriter<tokio::fs::File>`, `reqwest::async_impl::response::Response`, `impl futures::Future<Output = std::result::Result<reqwest::async_impl::response::Response, reqwest::error::Error>>`, `std::option::Option<actix_web::web::Bytes>`, `std::ops::ControlFlow<std::result::Result<std::convert::Infallible, anyhow::Error>, std::option::Option<actix_web::web::Bytes>>`, `impl futures::Future<Output = std::result::Result<std::option::Option<actix_web::web::Bytes>, reqwest::error::Error>>`, `actix_web::web::Bytes`, `tokio::io::util::write::Write<'_, tokio::io::BufWriter<tokio::fs::File>>`, `tokio::io::util::flush::Flush<'_, tokio::io::BufWriter<tokio::fs::File>>`
required because it captures the following types: `std::string::String`, `std::path::PathBuf`, `std::path::PathBuf`, `impl futures::Future<Output = std::result::Result<(), anyhow::Error>>`, `impl futures::Future<Output = std::result::Result<(), anyhow::Error>>`
required because it captures the following types: `&chromiumoxide_fetcher::browser::fetcher::BrowserFetcher`, `impl futures::Future<Output = bool>`, `impl futures::Future<Output = std::result::Result<(), chromiumoxide_fetcher::error::FetcherError>>`
required because it captures the following types: `tokio::sync::RwLockWriteGuard<'_, std::option::Option<proxy_domain::Browser>>`, `std::path::PathBuf`, `impl futures::Future<Output = std::result::Result<(), std::io::Error>>`, `chromiumoxide_fetcher::browser::fetcher::BrowserFetcher`, `chromiumoxide_fetcher::browser::revision_info::BrowserFetcherRevisionInfo`, `impl futures::Future<Output = std::result::Result<chromiumoxide_fetcher::browser::revision_info::BrowserFetcherRevisionInfo, chromiumoxide_fetcher::error::FetcherError>>`, `proxy_domain::Browser`, `impl futures::Future<Output = std::result::Result<proxy_domain::ConfigEntry<&proxy_domain::Browser>, proxy_services::ConfigServiceError>>`
required because it captures the following types: `tokio::sync::RwLockWriteGuard<'_, std::option::Option<proxy_domain::Browser>>`, `impl futures::Future<Output = tokio::sync::RwLockWriteGuard<'static, std::option::Option<proxy_domain::Browser>>>`, `impl futures::Future<Output = std::result::Result<proxy_domain::Browser, proxy_services::RenderServiceError>>`
required because it captures the following types: `impl futures::Future<Output = std::result::Result<proxy_domain::Browser, proxy_services::RenderServiceError>>`
required because it captures the following types: `helpers::authentication::guard::AuthenticationGuard`, `impl futures::Future<Output = std::result::Result<(), async_graphql::Error>>`, `impl futures::Future<Output = std::result::Result<graphql::resolvers::mutations::browser::InstallBrowserPayload, async_graphql::Error>>`
required because it captures the following types: `&async_graphql::ContextBase<'_, &async_graphql::Positioned<async_graphql::async_graphql_parser::types::Field>>`, `graphql::resolvers::mutations::browser::InstallBrowserPayload`, `{async block@proxy-api/src/graphql/resolvers/mutations/browser.rs:35:1: 35:10}`, `async_graphql::ContextBase<'_, &async_graphql::Positioned<async_graphql::async_graphql_parser::types::SelectionSet>>`, `impl futures::Future<Output = std::result::Result<async_graphql::Value, async_graphql::ServerError>>`
required because it captures the following types: `&async_graphql::ContextBase<'_, &async_graphql::Positioned<async_graphql::async_graphql_parser::types::Field>>`, `impl futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>>`, `impl futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>>`, `impl futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>>`
required because it captures the following types: `impl futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>> + std::marker::Send`
required because it captures the following types: `&async_graphql::MergedObject<&graphql::resolvers::mutations::browser::BrowserRootMutation, async_graphql::MergedObject<&graphql::resolvers::mutations::backup::BackupRootMutation, async_graphql::MergedObject<&graphql::resolvers::mutations::automate_task::AutomateTaskRootMutation, async_graphql::MergedObject<&graphql::resolvers::mutations::automate_session::AutomateSessionRootMutation, async_graphql::MergedObject<&graphql::resolvers::mutations::automate_entry::AutomateEntryRootMutation, async_graphql::MergedObject<&graphql::resolvers::mutations::authentication::AuthenticationRootMutation, async_graphql::MergedObject<&graphql::resolvers::mutations::assistant::AssistantRootMutation, async_graphql::MergedObjectTail>>>>>>>`, `&async_graphql::ContextBase<'_, &async_graphql::Positioned<async_graphql::async_graphql_parser::types::Field>>`, `std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>`, `impl futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>> + std::marker::Send`, `impl futures::Future<Output = std::result::Result<std::option::Option<async_graphql::Value>, async_graphql::ServerError>> + std::marker::Send`

The message is longer but contains more our internal stuff

Steps to Reproduce the Problem

  1. Create a large merged object (40-50 merged items) that uses guards

Specifications

  • Version: 7.0.2
  • Platform: OSX
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Sytten