comparison mercurial/hgweb/hgweb_mod.py @ 36801:66de4555cefd

wireproto: formalize permissions checking as part of protocol interface

Per the inline comment desiring to formalize permissions checking
in the protocol interface, we do that.

I'm not convinced this is the best way to go about things. I would love
for there to e.g. be a better exception for denoting permissions
problems. But it does feel strictly better than snipping attributes
on the proto instance.

Differential Revision: https://phab.mercurial-scm.org/D2719

36800:0b18604db95e

            query = req.env[r'QUERY_STRING'].partition(r'&')[0]
            query = query.partition(r';')[0]

        # Route it to a wire protocol handler if it looks like a wire protocol
        # request.
        protohandler = wireprotoserver.parsehttprequest(rctx.repo, req, query)

        if protohandler:
            try:
                if query:
                    raise ErrorResponse(HTTP_NOT_FOUND)

                # TODO fold this into parsehttprequest
                checkperm = lambda op: self.check_perm(rctx, req, op)
                protohandler['proto'].checkperm = checkperm

                return protohandler['dispatch'](checkperm)
            except ErrorResponse as inst:
                return protohandler['handleerror'](inst)

        # translate user-visible url structure to internal structure

36801:66de4555cefd

            query = req.env[r'QUERY_STRING'].partition(r'&')[0]
            query = query.partition(r';')[0]

        # Route it to a wire protocol handler if it looks like a wire protocol
        # request.
        protohandler = wireprotoserver.parsehttprequest(rctx, req, query,
                                                        self.check_perm)

        if protohandler:
            try:
                if query:
                    raise ErrorResponse(HTTP_NOT_FOUND)

                return protohandler['dispatch']()
            except ErrorResponse as inst:
                return protohandler['handleerror'](inst)

        # translate user-visible url structure to internal structure