← Back to branch summary

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

« back to all changes in this revision

Viewing changes to lib/common/interpret.py

  • Committer: wagrant
  • Date: 2008-12-23 01:58:44 UTC
  • Revision ID: svn-v3-trunk0:2b9c9e99-6f39-0410-b283-7f802c844ae2:trunk:1063
Set PATH_INFO and PATH_TRANSLATED properly for student CGI scripts.

If the requested path doesn't exist, interpretservice now traverses up the
directory tree until it finds a prefix of the path that does exist. If we are
allowed to execute that path, we do so, giving it the trimmed bits in
PATH_INFO. If execution is not permissible, we 404 anyway.

Fixes [ 2094777 ] Serve: Allow path after CGI script

Show diffs side-by-side

added added

removed removed

Lines of Context:
lib/common/interpret.py
409
409
 
410
410
    # PATH_INFO is wrong because the script doesn't physically exist.
411
411
    # Apache makes it relative to the "serve" app. It should actually be made
412
 
    # relative to the student's script.
413
 
    # TODO: At this stage, it is not possible to add a path after the script,
414
 
    # so PATH_INFO is always "".
415
 
    path_info = ""
416
 
    env['PATH_INFO'] = path_info
417
 
 
418
 
    # PATH_TRANSLATED currently points to a non-existant location within the
419
 
    # local web server directory. Instead make it represent a path within the
420
 
    # student jail.
421
 
    (username, _, path_translated) = studpath.url_to_jailpaths(req.path)
422
 
    if len(path_translated) == 0 or path_translated[0] != os.sep:
423
 
        path_translated = os.sep + path_translated
424
 
    env['PATH_TRANSLATED'] = path_translated
 
412
    # relative to the student's script. intepretservice does that in the jail,
 
413
    # so here we just clear it.
 
414
    env['PATH_INFO'] = ''
 
415
    env['PATH_TRANSLATED'] = ''
425
416
 
426
417
    # CGI specifies that REMOTE_HOST SHOULD be set, and MAY just be set to
427
418
    # REMOTE_ADDR. Since Apache does not appear to set this, set it to
431
422
 
432
423
    # SCRIPT_NAME is the path to the script WITHOUT PATH_INFO.
433
424
    script_name = req.uri
434
 
    if len(path_info) > 0:
435
 
        script_name = script_name[:-len(path_info)]
436
425
    env['SCRIPT_NAME'] = script_name
437
426
 
438
427
    # SERVER_SOFTWARE is actually not Apache but IVLE, since we are
440
429
    env['SERVER_SOFTWARE'] = "IVLE/" + str(conf.ivle_version)
441
430
 
442
431
    # Additional environment variables
 
432
    username = studpath.url_to_jailpaths(req.path)[0]
443
433
    env['HOME'] = os.path.join('/home', username)