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

« back to all changes in this revision

Viewing changes to lib/common/interpret.py

  • Committer: mattgiuca
  • Date: 2008-07-21 04:21:18 UTC
  • Revision ID: svn-v3-trunk0:2b9c9e99-6f39-0410-b283-7f802c844ae2:trunk:917
interpret.py: Removed the code which removes HTTP_COOKIE from the CGI
    environment. Student code can now access cookies.
Note: This was previously a security risk because malicious code could
steal IVLE cookies. Now that we have separate domain space for other users,
the worst you can do is:
    a) Steal your own IVLE cookie.
    b) Steal other user's non-IVLE cookies (ie. other public cookies).

This makes all student code vulnerable to cookie theft, but that is simply a
disclaimer (if you use cookies, your apps are vulnerable). It is not a
security risk to IVLE itself.

Show diffs side-by-side

added added

removed removed

Lines of Context:
377
377
    # Comments here are on the heavy side, explained carefully for security
378
378
    # reasons. Please read carefully before making changes.
379
379
 
380
 
    # Remove HTTP_COOKIE. It is a security risk to have students see the IVLE
381
 
    # cookie of their visitors.
382
 
    try:
383
 
        del env['HTTP_COOKIE']
384
 
    except: pass
385
 
 
386
380
    # Remove DOCUMENT_ROOT and SCRIPT_FILENAME. Not part of CGI spec and
387
381
    # exposes unnecessary details about server.
388
382
    try: