~azzar1/unity/add-show-desktop-key

« back to all changes in this revision

Viewing changes to ivle/dispatch/__init__.py

  • Committer: William Grant
  • Date: 2009-12-02 02:20:57 UTC
  • mto: This revision was merged to the branch mainline in revision 1353.
  • Revision ID: grantw@unimelb.edu.au-20091202022057-m3w3rzrzp47y89to
Refuse to traverse through an object to which the user has no permissions. This stops information leakage in breadcrumbs.

Show diffs side-by-side

added added

removed removed

Lines of Context:
37
37
import time
38
38
 
39
39
import mod_python
40
 
import routes
41
40
 
42
41
from ivle import util
43
42
import ivle.config
46
45
from ivle.webapp.base.plugins import ViewPlugin, PublicViewPlugin
47
46
from ivle.webapp.base.xhtml import XHTMLView, XHTMLErrorView
48
47
from ivle.webapp.errors import HTTPError, Unauthorized, NotFound
 
48
from ivle.webapp.publisher import Publisher, PublishingError
 
49
from ivle.webapp import ApplicationRoot
49
50
 
50
51
config = ivle.config.Config()
51
52
 
52
 
def generate_router(view_plugins, attr):
 
53
class ObjectPermissionCheckingPublisher(Publisher):
 
54
    """A specialised publisher that checks object permissions.
 
55
 
 
56
    This publisher verifies that the user holds any permission at all
 
57
    on the model objects through which the resolution path passes. If
 
58
    no permission is held, resolution is aborted with an Unauthorized
 
59
    exception.
 
60
 
 
61
    IMPORTANT: This does NOT check view permissions. It only checks
 
62
    the objects in between the root and the view, exclusive!
 
63
    """
 
64
 
 
65
    def traversed_to_object(self, obj):
 
66
        """Check that the user has any permission at all over the object."""
 
67
        if (hasattr(obj, 'get_permissions') and
 
68
            len(obj.get_permissions(self.root.user)) == 0):
 
69
            raise Unauthorized()
 
70
 
 
71
 
 
72
def generate_publisher(view_plugins, root):
53
73
    """
54
74
    Build a Mapper object for doing URL matching using 'routes', based on the
55
75
    given plugin registry.
56
76
    """
57
 
    m = routes.Mapper(explicit=True)
 
77
    r = ObjectPermissionCheckingPublisher(root=root)
 
78
 
 
79
    r.add_set_switch('api', 'api')
 
80
 
58
81
    for plugin in view_plugins:
59
 
        # Establish a URL pattern for each element of plugin.urls
60
 
        assert hasattr(plugin, 'urls'), "%r does not have any urls" % plugin 
61
 
        for url in getattr(plugin, attr):
62
 
            routex = url[0]
63
 
            view_class = url[1]
64
 
            kwargs_dict = url[2] if len(url) >= 3 else {}
65
 
            m.connect(routex, view=view_class, **kwargs_dict)
66
 
    return m
 
82
        if hasattr(plugin, 'forward_routes'):
 
83
            for fr in plugin.forward_routes:
 
84
                # An annotated function can also be passed in directly.
 
85
                if hasattr(fr, '_forward_route_meta'):
 
86
                    r.add_forward_func(fr)
 
87
                else:
 
88
                    r.add_forward(*fr)
 
89
 
 
90
        if hasattr(plugin, 'reverse_routes'):
 
91
            for rr in plugin.reverse_routes:
 
92
                # An annotated function can also be passed in directly.
 
93
                if hasattr(rr, '_reverse_route_src'):
 
94
                    r.add_reverse_func(rr)
 
95
                else:
 
96
                    r.add_reverse(*rr)
 
97
 
 
98
        if hasattr(plugin, 'views'):
 
99
            for v in plugin.views:
 
100
                r.add_view(*v)
 
101
 
 
102
    return r
67
103
 
68
104
def handler(apachereq):
69
105
    """Handles an HTTP request.
85
121
            req.user = user
86
122
 
87
123
    if req.publicmode:
88
 
        req.mapper = generate_router(config.plugin_index[PublicViewPlugin],
89
 
                                    'public_urls')
90
 
    else:
91
 
        req.mapper = generate_router(config.plugin_index[ViewPlugin], 'urls')
92
 
 
93
 
    matchdict = req.mapper.match(req.uri)
94
 
    if matchdict is not None:
95
 
        viewcls = matchdict['view']
96
 
        # Get the remaining arguments, less 'view', 'action' and 'controller'
97
 
        # (The latter two seem to be built-in, and we don't want them).
98
 
        kwargs = matchdict.copy()
99
 
        del kwargs['view']
 
124
        raise NotImplementedError("no public mode with obtrav yet!")
 
125
 
 
126
    req.publisher = generate_publisher(
 
127
        config.plugin_index[ViewPlugin],
 
128
        ApplicationRoot(req.config, req.store, req.user))
 
129
 
 
130
    try:
 
131
        obj, viewcls, subpath = req.publisher.resolve(req.uri.decode('utf-8'))
100
132
        try:
 
133
            # We 404 if we have a subpath but the view forbids it.
 
134
            if not viewcls.subpath_allowed and subpath:
 
135
                raise NotFound()
 
136
 
101
137
            # Instantiate the view, which should be a BaseView class
102
 
            view = viewcls(req, **kwargs)
 
138
            view = viewcls(req, obj, subpath)
103
139
 
104
140
            # Check that the request (mainly the user) is permitted to access
105
141
            # the view.
118
154
                errviewcls = XHTMLView.get_error_view(e)
119
155
 
120
156
            if errviewcls:
121
 
                errview = errviewcls(req, e)
 
157
                errview = errviewcls(req, e, obj)
122
158
                errview.render(req)
123
159
                return req.OK
124
160
            elif e.message:
138
174
        else:
139
175
            req.store.commit()
140
176
            return req.OK
141
 
    else:
142
 
        XHTMLErrorView(req, NotFound()).render(req)
 
177
    except Unauthorized, e:
 
178
        # Resolution failed due to a permission check. Display a pretty
 
179
        # error, or maybe a login page.
 
180
        XHTMLView.get_error_view(e)(req, e, req.publisher.root).render(req)
 
181
        return req.OK
 
182
    except PublishingError, e:
 
183
        req.status = 404
 
184
 
 
185
        if req.user and req.user.admin:
 
186
            XHTMLErrorView(req, NotFound('Not found: ' +
 
187
                                         str(e.args)), e[0]).render(req)
 
188
        else:
 
189
            XHTMLErrorView(req, NotFound(), e[0]).render(req)
 
190
 
143
191
        return req.OK
144
192
 
145
193
def handle_unknown_exception(req, exc_type, exc_value, exc_traceback):
155
203
    req.content_type = "text/html"
156
204
    logfile = os.path.join(config['paths']['logs'], 'ivle_error.log')
157
205
    logfail = False
 
206
 
 
207
    # XXX: This remains here for ivle.interpret's IVLEErrors. Once we rewrite
 
208
    #      fileservice, req.status should always be 500 (ISE) here.
158
209
    try:
159
210
        httpcode = exc_value.httpcode
160
211
        req.status = httpcode
161
212
    except AttributeError:
162
213
        httpcode = None
163
214
        req.status = mod_python.apache.HTTP_INTERNAL_SERVER_ERROR
 
215
 
164
216
    try:
165
217
        publicmode = req.publicmode
166
218
    except AttributeError:
189
241
    # misbehaves (which is currently very easy, if things aren't set up
190
242
    # correctly).
191
243
    # Write the traceback.
192
 
    # If this is a non-4xx IVLEError, get the message and httpcode and
193
 
    # make the error message a bit nicer (but still include the
194
 
    # traceback).
195
 
    # We also need to special-case IVLEJailError, as we can get another
 
244
 
 
245
    # We need to special-case IVLEJailError, as we can get another
196
246
    # almost-exception out of it.
197
 
 
198
 
    codename, msg = None, None
199
 
 
200
247
    if exc_type is util.IVLEJailError:
201
 
        msg = exc_value.type_str + ": " + exc_value.message
202
248
        tb = 'Exception information extracted from IVLEJailError:\n'
203
249
        tb += urllib.unquote(exc_value.info)
204
250
    else:
205
 
        try:
206
 
            codename, msg = req.get_http_codename(httpcode)
207
 
        except AttributeError:
208
 
            pass
209
 
 
210
251
        tb = ''.join(traceback.format_exception(exc_type, exc_value,
211
252
                                                exc_traceback))
212
253
 
213
 
    logging.error('%s\n%s'%(str(msg), tb))
 
254
    logging.error('\n' + tb)
214
255
 
215
256
    # Error messages are only displayed is the user is NOT a student,
216
257
    # or if there has been a problem logging the error message
220
261
<html xmlns="http://www.w3.org/1999/xhtml">
221
262
<head><title>IVLE Internal Server Error</title></head>
222
263
<body>
223
 
<h1>IVLE Internal Server Error""")
224
 
    if show_errors:
225
 
        if codename is not None and \
226
 
           httpcode != mod_python.apache.HTTP_INTERNAL_SERVER_ERROR:
227
 
            req.write(": %s" % cgi.escape(codename))
228
 
 
229
 
    req.write("""</h1>
 
264
<h1>IVLE Internal Server Error</h1>
230
265
<p>An error has occured which is the fault of the IVLE developers or
231
266
administrators. """)
232
267
 
238
273
    req.write("</p>")
239
274
 
240
275
    if show_errors:
241
 
        if msg is not None:
242
 
            req.write("<p>%s</p>\n" % cgi.escape(msg))
243
 
        if httpcode is not None:
244
 
            req.write("<p>(HTTP error code %d)</p>\n" % httpcode)
245
276
        req.write("<h2>Debugging information</h2>")
246
 
 
247
277
        req.write("<pre>\n%s\n</pre>\n"%cgi.escape(tb))
248
278
    req.write("</body></html>")