How to set Byte payload config

[stevemk14ebr]

I cannot figure out how to set the payload limit for the Bytes type. I'm trying to use

async fn view_index_features(state: ActixAppState, _: HttpRequest, body: Bytes) -> Result<HttpResponse, Error> {
 ...blah...

}

App::new()
     .wrap(middleware::Logger::default())
     .app_data(app_state.clone())
     .service(web::resource("/index/")
         .app_data(web::PayloadConfig::new(1000000 * 250))
         .route(web::put().to(view_index_features)))

But this gives an app_data is not configured error. If i do .data() on the app itself the payload limit is not respected and i get a 413: A payload reached size limit. I'm on the latest actix 2 version

[stevemk14ebr]

Wait... can you not have a stateful app, and also configure route upload settings? I got it working if i do the app_data at the App level but it doesn't work at the service level. How do i get different configs at the service level

[Jonathas-Conceicao]

Could you try using FromRequest::configure for it? Something like:

.app_data(actix_web::web::Bytes::configure(|cfg| {
    cfg.limit(1000000 * 250) // Set max file size to 6 Mib
}))

Documentation for JsonConfigure has a more complete example for it.

[stevemk14ebr]

I get the same error about app data not being configured if i do that. I believe this is because service app_data overrides app level app_data but i am not sure. I have to add the config as an app level config but that effects all routes.

[stevemk14ebr]

@robjtede i know you did some work with this data layer stuff. Is the behavior i am seeing expected. A working configuration is:

    let srv = HttpServer::new(move || {
            App::new()
                .wrap(NormalizeSlash)
                .wrap(middleware::Logger::default())
                .wrap(Cors::default())
                .app_data(app_state.clone())
                .app_data(web::PayloadConfig::new(1000000 * 250))
                .app_data(web::Json::<ViewMultFeatureSamplesInput>::configure(|cfg| {
                    cfg.limit(1000000 * 25).error_handler(json_error_handler) 
                }))

The json config must be specified at the top level .app_data. If i move it to any route below i get a runtime error about data not being configured and it appears to 'unregister' my app_state.clone() data line.

[cybuch]

If you call HTTP client that will have response larger than 256kb then you should also set limit for that response e.g.

            request.send()
                .await
                .map_err(Error::from)?
                .body()
                .limit(1024)

[robjtede]

Just following up @stevemk14ebr, this might be solved by the changes made to data stacking in v3 beta.

[robjtede]

I believe this issue is fixed. Please open a new issue if further problems arise.

[kpcyrd]

I've opened a discussion on how to do this with actix-web 4: #2722