← Back to branch summary

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

« back to all changes in this revision

Viewing changes to ivle/webapp/console/service.py

Error views are now retrieved from a class method of the view, not the plugin
registry. They also now discriminate based on exception type.

Show diffs side-by-side

added added

removed removed

Lines of Context:
ivle/webapp/console/service.py
31
31
 
32
32
import ivle.console
33
33
import ivle.chat
 
34
import ivle.conf
34
35
from ivle.webapp.base.rest import JSONRESTView, named_operation
35
36
from ivle.webapp.errors import BadRequest
36
37
 
37
38
# XXX: Should be RPC view, with actions in URL?
38
39
class ConsoleServiceRESTView(JSONRESTView):
39
40
    '''An RPC interface to a Python console.'''
40
 
    def get_permissions(self, user, config):
41
 
        if user is not None:
42
 
            return set(['use'])
43
 
        else:
44
 
            return set()
45
 
 
46
 
    @named_operation('use')
 
41
    @named_operation
47
42
    def start(self, req, cwd=''):
48
43
        working_dir = os.path.join("/home", req.user.login, cwd)
49
44
 
50
45
        uid = req.user.unixid
51
46
 
52
47
        # Start the server
53
 
        jail_path = os.path.join(req.config['paths']['jails']['mounts'],
54
 
                                 req.user.login)
55
 
        cons = ivle.console.Console(req.config, uid, jail_path, working_dir)
 
48
        jail_path = os.path.join(ivle.conf.jail_base, req.user.login)
 
49
        cons = ivle.console.Console(uid, jail_path, working_dir)
56
50
 
57
51
        # Assemble the key and return it. Yes, it is double-encoded.
58
52
        return {'key': cjson.encode({"host": cons.host,
59
53
                                     "port": cons.port,
60
54
                                     "magic": cons.magic}).encode('hex')}
61
55
 
62
 
    @named_operation('use')
63
 
    def chat(self, req, key, text='', cwd='', kind="chat"):
 
56
    @named_operation
 
57
    def chat(self, req, key, text='', kind="chat"):
64
58
        # The request *should* have the following four fields:
65
59
        # key: Hex JSON dict of host and port where the console server lives,
66
60
        # and the secret to use to digitally sign the communication with the
76
70
        except KeyError:
77
71
            raise BadRequest("Invalid console key.")
78
72
 
79
 
        jail_path = os.path.join(req.config['paths']['jails']['mounts'],
80
 
                                 req.user.login)
81
 
        # Within Jail
82
 
        working_dir = os.path.join("/home", req.user.login, cwd)
 
73
        jail_path = os.path.join(ivle.conf.jail_base, req.user.login)
 
74
        working_dir = os.path.join("/home", req.user.login)   # Within jail
83
75
        uid = req.user.unixid
84
76
 
85
 
        # XXX: JSONRESTView should do this for us.
86
 
        text = text.decode('utf-8')
87
 
 
88
77
        msg = {'cmd':kind, 'text':text}
89
78
        try:
 
79
            json_response = ivle.chat.chat(host, port, msg, magic,decode=False)
 
80
 
 
81
            # Snoop the response from python-console to check that it's valid
90
82
            try:
91
 
                json_response = ivle.chat.chat(host, port, msg, magic,decode=False)
92
 
                # Snoop the response from python-console to check that it's valid
93
83
                response = cjson.decode(json_response)
94
 
            except (cjson.DecodeError, ivle.chat.ProtocolError):
 
84
            except cjson.DecodeError:
95
85
                # Could not decode the reply from the python-console server
96
86
                response = {"terminate":
97
87
                    "Communication to console process lost"}
98
88
            if "terminate" in response:
99
 
                response = restart_console(req.config, uid, jail_path,
100
 
                    working_dir, response["terminate"])
 
89
                response = restart_console(uid, jail_path, working_dir,
 
90
                    response["terminate"])
101
91
        except socket.error, (enumber, estring):
102
92
            if enumber == errno.ECONNREFUSED:
103
93
                # Timeout: Restart the session
104
 
                response = restart_console(req.config, uid, jail_path,
105
 
                    working_dir,
 
94
                response = restart_console(uid, jail_path, working_dir,
106
95
                    "The IVLE console has timed out due to inactivity")
107
96
            elif enumber == errno.ECONNRESET:
108
97
                # Communication issue: Restart the session
109
 
                response = restart_console(req.config, uid, jail_path,
110
 
                    working_dir,
 
98
                response = restart_console(uid, jail_path, working_dir,
111
99
                    "Connection with the console has been reset")
112
100
            else:
113
101
                # Some other error - probably serious
115
103
        return response
116
104
 
117
105
 
118
 
def restart_console(config, uid, jail_path, working_dir, reason):
 
106
def restart_console(uid, jail_path, working_dir, reason):
119
107
    """Tells the client that it must be issued a new console since the old 
120
108
    console is no longer availible. The client must accept the new key.
121
109
    Returns the JSON response to be given to the client.
122
110
    """
123
111
    # Start a new console server console
124
 
    cons = ivle.console.Console(config, uid, jail_path, working_dir)
 
112
    cons = ivle.console.Console(uid, jail_path, working_dir)
125
113
 
126
114
    # Make a JSON object to tell the browser to restart its console client
127
115
    new_key = cjson.encode(