Cornices/cornice

Why add 'context' in resource factory init parameters ?

olemoign opened this issue · 3 comments

params['context'] = request.context

This breaks the Pyramid pattern of expecting only the request in the init() function and I don't really see the point of copying the request.context that will anyway be present ?

This has been here for a few years now, introduced by @mardiros in #209

It raised some questions while we refactored the way Cornice handles ACL. You can read @wjehenddher interrogations here:
https://github.com/Cornices/cornice/pull/456/files#r138968777

If you have a proposal, please share it! We really would like to align Cornice design with Pyramid, even if it means introducing other breaking changes.

@olemoign would you be interested in fixing this?

The project I was working on with cornice has shutdown so I won't be able to. Sorry.