Consider disallowing root nesting

[jplatte]

Before the 0.8.0 release, I would like to come back to nest("/", router) and nest_service("/", svc). I believe I suggested disallowing them (in favor of merge and fallback_service) before, but if so there must have been some pushback by David since we still allow both.

Any pointers or thoughts appreciated.

[jplatte]

The only thing I found is this: #2012
… which suggests I'm wrong about previous push-back and we should just disallow root nesting :)

[dancixx]

@jplatte are you planning to deprecate the nest function or just improve it? This is very handy and makes the code much readable.

[jplatte]

@dancixx This issue is specifically about nest and nest_service calls where the path (first argument) is /.

[dancixx]

@dancixx This issue is specifically about nest and nest_service calls where the path (first argument) is /.

@jplatte yes Im talking about this: Route::new().nest("/", Router::new()...). But if this issue is not about deprecating this feature rather then just make it more conveniently; never mind.

[jplatte]

Is there any reason you use that rather than Router::merge?

[dancixx]

@jplatte I don't have any specific reason I'm just more familiar with the nest function. Actually, I started a new project using nest and I have found this issue and I don't want to use deprecated features, that was main the reason why I asked it.

[jplatte]

Yeah, I would recommend using merge(...) instead of nest("/", ...) then :)
It should be equivalent in functionality.