Cornices/cornice

Why was traversal parameter support removed?

Closed this issue · 4 comments

dz0ny commented

I have seen #179 (comment) but later versions added support for traverse parameter, however 3.2 removed support? Was it for performance reasons or something else? I've see colander_path_validator but that has different use case...

Good question!

I arrived late in the Cornice project, and @tarekziade and @almet always let me assume that traversal was not supported.

Since I never considered it supported, its removal in 3.2 should be a side-effect and not intentional!

If you're interested, we can restore it, and the best for that would be that you contribute a small example setup as a test suite so that it becomes part of the expected features :)

Thanks for your feedback!

If you're interested, we can restore it, and the best for that would be that you contribute a small example setup as a test suite so that it becomes part of the expected features :)

@leplatrem I've created a PR for this. Please check it out and ping me if I should change anything.

Fixed in #484

I think we can close this.