405 Commits (a4366e86f7459ce166ce71012e2e81e323ef21f4)

Author	SHA1	Message	Date
Min RK	81f8cfeccb	cleanup some html services tests	11 years ago
Min RK	5786346bde	Merge pull request #8168 from quantopian/stop-delete-root ... BUG: Don't allow ContentsManager to delete root.	11 years ago
Scott Sanderson	f2825fdcad	BUG: Don't allow ContentsManager to delete root. ... Doing so leaves the notebook app in a bad state.	11 years ago
Min RK	02739afa42	use >= instead of > when comparing dates ... fast test runners can result in equal values.	11 years ago
Min RK	849e752d3d	more profile awareness from config.manager to IPython subclass ... also enables specifying profile by name, instead of full path.	11 years ago
Scott Sanderson	e8fd4ba9c2	TEST: Allow greater or equal modified_date on rename.	11 years ago
Scott Sanderson	72aeb78aaf	TEST: Add test for handling of last_modified. ... Saving should increment last_modified. Renaming should not. Noticed because incrementing last_modified on rename results in a scary error message from the frontend.	11 years ago
Scott Sanderson	7971a5ffc7	TEST: Also test deletion.	11 years ago
Scott Sanderson	fdcbd02ffb	TEST: Add test for renaming files with checkpoint. ... The checkpoint name should follow the file rename.	11 years ago
Min RK	9ded491c97	Merge pull request #7708 from SylvainCorlay/allow_none ... allow_none=False for all trait types but Instance and Type	11 years ago
Scott Sanderson	e21b99900d	MAINT: Remove unused import.	11 years ago
Scott Sanderson	257c7296c9	TEST: Move test_escape_root to TestFileContentsManager ... `root_dir` is specific to the FileContentsManager implementation.	11 years ago
Min RK	62e4a5d072	contents: double check that os_path is always within root ... check based on tornado `StaticFileHandler.validate_absolute_path` I haven't been able to actually escape root, but we should have this check anyway.	11 years ago
Thomas Kluyver	2b95955079	Merge pull request #7906 from minrk/skip-perm-test-windows ... skip permission error -> 403 test on Windows	11 years ago
Matthias Bussonnier	709126542a	Merge pull request #7832 from minrk/default-secure-session ... remove default_secure, make Session secure by default	11 years ago
Min RK	0bc4549d25	skip permission -> 403 test on Windows ... The test actually passes on my VM (Win 7), but not on Jenkins (Server 2012). I haven't figured out how to identify the subset of Windows systems where it won't work, but since the problem appears to be in the test, not the tested code, skipping on Windows seems the right way to go.	11 years ago
Min RK	d9f01671ae	make it clearer that we adapt to old messages ... > Adapting to protocol vX for kernel Y instead of > Kernel Y speaks protocol version X some have interpreted the "Kernel speaks" statement was an error message; this should be clearer about what's happening, and that there is no error.	11 years ago
Min RK	109ee00ebb	get session.key from kernel manager ... in kernel channel handlers we were accidentally relying on loading Session.key from config set via the default_secure decorator, which forced the key to be the same for every kernel.	11 years ago
Fernando Perez	50c8c5feed	Merge pull request #7852 from minrk/open-perm ... Catch IOError in addition to OSError. Closes #7848	11 years ago
Thomas Kluyver	80b931897c	Fix some formatting for config options help	11 years ago
Thomas Kluyver	0dd26a9b32	Merge pull request #7855 from minrk/no-extra-contents-manager ... don't force instantiation of extra, unused contents manager	11 years ago
Min RK	ce7e03bc61	make sure SessionManager tests have a ContentsManager	11 years ago
Thomas Kluyver	193ecc9d22	Merge pull request #7858 from takluyver/i7857 ... Dump frontend config JSON with an indent	11 years ago
Thomas Kluyver	8e4514d88b	Dump frontend config JSON with an indent ... Fixes gh-7857	11 years ago
Min RK	94656735f3	don't force instantiation of contents manager by default ... this is never used and never needed, and results in the instantiation of a contents manager that is immediately discarded.	11 years ago
Min RK	b83a9383a6	test that FileContentsManager.open turns permission error into 403	11 years ago
Min RK	4e6805c408	catch IOError in addition to OSError ... IOError is a subclass of OSError on py3, but not py2	11 years ago
Jason Grout	512187ac10	Fix contentsmanager api error message to give the path name	11 years ago
Sylvain Corlay	5c16ffe559	removing redundant allow_none=False	11 years ago
Min RK	121d495424	protect websocket against errant messages ... - message handler being triggered after stream is closed (no channels) - messages sent to invalid channels	11 years ago
Min RK	ad83e79ea5	allow requesting contents without body ... adds `?content=0` to only fetch the metadata in the model. The REST API now exposes all arguments to ContentsManager.get.	11 years ago
Brian E. Granger	ab83599d36	Merge pull request #7389 from minrk/one-websocket ... use single WebSocket connection for all channels	11 years ago
Min RK	3cf6eacc86	Merge pull request #7324 from quantopian/separate-checkpoint-manager ... DEV: Refactor checkpoint logic out of FileContentsManager into a separate class.	11 years ago
Scott Sanderson	64e5c49608	DEV: Break get_checkpoint into separate methods. ... One for notebooks and one for checkpoints.	11 years ago
Scott Sanderson	2569c3069d	STY: s/CheckpointManager/Checkpoints ... It's easy to confuse CheckpointManager and ContentsManager.	11 years ago
Scott Sanderson	021e2da495	DEV: Separate FileCheckpointManager and GenericFileCheckpointManager. ... - Adds a `GenericCheckpointMixin` as a helper for implementing the two boundary-traversing Checkpoint API methods, `create_checkpoint` and `restore_checkpoint`. - `GenericFileCheckpointManager` is implemented as a subclass of `FileCheckpointManager` using `GenericCheckpointMixin`. Note that this is the safe subtyping relationship because of method signature *contra*variance: `FileCheckpointManager` accepts `FileContentsManager` in its method signatures type, whereas `GenericFileCheckpointManager` accepts any `ContentsManager`. - Moved Checkpoint-related classes to their own files.	11 years ago
Scott Sanderson	12fe97e2af	DEV: Allow CheckpointManagers to optimize for shared backends. ... On `create_checkpoint` and `restore_checkpoint`, pass a path and a `ContentsManager` to `CheckpointManager` instead of an already-loaded model. The `CheckpointManager` base class provides a correct implementation of these methods that's generic across any ContentsManager, but subclasses are free to specialize when the storage backend of `ContentsManager` is shared.	11 years ago
Scott Sanderson	55d4e20aaf	DEV: Remove ContentsManager.walk.	11 years ago
Scott Sanderson	2b73f1e620	DEV: Implement os.walk analog for ContentsManagers.	11 years ago
Scott Sanderson	1e2e86dcca	MAINT: Return dicts from CheckpointManager.get_checkpoint. ... The output is going to get converted to a dict anyway, and this makes it easier to pipe output from a CheckpointManager directly to a ContentsManager.	11 years ago
Min RK	1cac03528b	use single WebSocket connection for all channels ... multiplex on a 'channel' key in message, rather than separate websockets. Unlike zmq, there aren't different message patterns that require the channels to be separate. Reduces FD count by factor of 3 and connection complexity in js.	11 years ago
Scott Sanderson	7030a8717a	DEV: Add full support for non-notebook checkpoints.	11 years ago
Scott Sanderson	23837e9ad4	DEV: Remove unnecessary notary calls.	11 years ago
Min RK	67abf3ddab	update kernelspec test ... to expect Python display name	11 years ago
Scott Sanderson	631a68d83e	TEST: Test separate roots for Contents and Checkpoints.	11 years ago
Scott Sanderson	f71f216880	DEV: More checkpoint API refactoring. ... Changed the public checkpoint API to: - `create_checkpoint(self, nb, path)` - `get_checkpoint_content(self, checkpoint_id, path)` - `rename_checkpoint(self, checkpoint_id, old_path, new_path)` - `delete_checkpoint(self, checkpoint_id, path)` - `list_checkpoints(self, path)` All paths in the above are API-style paths, and the `nb` argument to `create_checkpoint` is a dictionary suitable for passing to `nbformat.write`. The new `get_checkpoint_content` method returns an unvalidated notebook content dictionary. It is used by `ContentManager.restore_checkpoint` to load content to be written via `save`.	11 years ago
Matthias Bussonnier	c1d9f4d436	Merge pull request #7283 from minrk/kernel-spec-model ... add resource URLs to kernelspec model	11 years ago
Scott Sanderson	dc295f67bd	MAINT: Add missing `config=True`s.	11 years ago
Scott Sanderson	4da8a30f20	DEV: Refactor checkpoint logic from FileContentsManager. ... - Add a `CheckpointManager` base class and infrastructure for creating a `checkpoint_manager` instance attribute on `ContentsManager`. - Provide default implementations of `delete` and `rename` in the base `ContentsManager` class. `ContentsManager` subclasses are now required to implement `delete_file` and `rename_file`. These methods no longer need to manage checkpoints. - Move checkpoint-related functionality from `FileContentsManager` to a dedicated `FileCheckpointManager` subclass. - Move shared filesystem interaction logic into `FileManagerMixin` used by both `FileContentsManager` and `FileCheckpointManager`. - Minor tweaks to ContentsManager tests to get methods from the right object. The purpose of this change is to provide an API for users to replace just the checkpoint logic associated with a particular `ContentsManager`. In particular, this change makes it possible to easily support remote storage of checkpoints while otherwise retaining normal filesystem interactions.	11 years ago
Scott Sanderson	9aad3c631f	MAINT: Move `update` from FileContentsManager to ContentsManager. ... The implementation in FileContentsManager is written entirely in terms of the required API, and is sensible for other ContentsManager implementations.	11 years ago