aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | Adds the unit tests and removes useless function we don't useJessica T2013-04-122-7/+57
| | | | | |
| * | | | | Adds the timesince ability which fixes #394Jessica T2013-04-115-3/+133
| | | | | |
* | | | | | adding callable_runone and callable_runall to the docsChristopher Allan Webber2013-04-111-1/+2
| | | | | |
* | | | | | Merge branch '637_friendlier_hooks'Christopher Allan Webber2013-04-109-10/+315
|\ \ \ \ \ \
| * | | | | | Switch several functions over to using the new callable_run* tools!Christopher Allan Webber2013-04-103-10/+5
| | | | | | |
| * | | | | | Remove unhandled_okay from kwargs if it's there before passing to functions.Christopher Allan Webber2013-04-101-1/+3
| | | | | | |
| * | | | | | callable hook convenience functions.. now work, and with tests!Christopher Allan Webber2013-04-106-5/+137
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Added three "callables" test plugins. - updated callable_runone to check for unhandled_okay in the kwargs dict. All passing!
| * | | | | | tests for new pluginapi methods that can't possibly pass yet :)Christopher Allan Webber2013-04-091-0/+111
| | | | | | |
| * | | | | | Convenience functions for callable hooksChristopher Allan Webber2013-04-071-0/+65
| | | | | | |
* | | | | | | Open Source -> Free Software from the borrowed extlib repositoriesChristopher Allan Webber2013-04-101-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | We're a GNU project, gotta get with the GNU world order ;)
* | | | | | Adding some help about what to do if flup flakes out on youChristopher Allan Webber2013-04-101-0/+5
| | | | | |
* | | | | | Fix sessions so they work across the site.Christopher Allan Webber2013-04-101-3/+1
| | | | | |
* | | | | | Make session cookies more secure.Elrond2013-04-092-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Our session cookies only need to be available to http, so mark them appropiately. 2. Send the cookie to the subpath for mediagoblin. And instantiate a session manager on the app, once.
* | | | | | Remove the last traces of beaker.Elrond2013-04-095-29/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There were still some traces of beaker around: - docs: replaced by reference to itsdangerous. - paste configs: Wiped away. - config_spec.ini: wiped. - test_mgoblin_app.ini: also wiped.
* | | | | | Fix left over from variable renaming.Elrond2013-04-091-1/+4
| | | | | |
* | | | | | Merge remote-tracking branch 'brett/itsdangerous'Elrond2013-04-0912-80/+223
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * brett/itsdangerous: Call is_updated instead of testing it boolean. Harden It's Dangerous key management. First tests for the Session class. Set a starting value for session.send_new_cookie. Remove beaker stuff from the code. Delete the session cookie on an empty session. Back sessions with It's Dangerous. Improve fs security for itsdangerous secret. Docs for get_timed_signer_url. Basic itsdangerous infrastructure. Conflicts: mediagoblin/tests/test_cache.py
| * | | | | | Call is_updated instead of testing it boolean.Brett Smith2013-03-251-1/+1
| | | | | | |
| * | | | | | Harden It's Dangerous key management.Brett Smith2013-03-241-20/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous code was theoretically subject to timing attacks, where an attacker could read the key in between the time it was saved to the file and when the chmod happened. This version prevents that by using umasks to ensure the files always have the right permissions. This version also avoids using a key that cannot be saved due to some system setup bug.
| * | | | | | First tests for the Session class.Brett Smith2013-03-241-0/+30
| | | | | | |
| * | | | | | Set a starting value for session.send_new_cookie.Brett Smith2013-03-241-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes session.__init__ slightly more complicated but probably simplifies everything else, especially if we make the class smarter later by having it track changes itself.
| * | | | | | Remove beaker stuff from the code.Brett Smith2013-03-247-79/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is all obsoleted by It's Dangerous.
| * | | | | | Delete the session cookie on an empty session.Brett Smith2013-03-241-1/+4
| | | | | | |
| * | | | | | Back sessions with It's Dangerous.Brett Smith2013-03-244-3/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a contribution to #668.
| * | | | | | Improve fs security for itsdangerous secret.Elrond2013-03-221-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Set mode 700 on the directory, mode 600 on the file.
| * | | | | | Docs for get_timed_signer_url.Elrond2013-03-221-0/+26
| | | | | | |
| * | | | | | Basic itsdangerous infrastructure.Elrond2013-03-223-0/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the basic infrastructure for using itsdangerous in mediagoblin. Usage instructions will follow.
* | | | | | | Use cascade for comment deletion.Elrond2013-04-081-7/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also use the relationship for getting the comments on a MediaEntry.
* | | | | | | Testing: Remove some left over files/dirs.Elrond2013-04-082-2/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using tempfile.* in testing, those files get created in the system tempdir. The docs say, we should try to remove them. Yes, the next reboot will clean them up also. And in the workbench case, check after each test, that the global workbench dir is empty (so the sub-workbcnhes have been destroyed).
* | | | | | | Implement queue dir delete for proc_state.delete_queue_fileElrond2013-04-081-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement queue dir deleting in the proc_state.delete_queue_file helper function.
* | | | | | | Merge remote-tracking branch 'cwebber/254_delete_queue_directories'Elrond2013-04-085-9/+57
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * cwebber/254_delete_queue_directories: Removing docstring bit about delete_file possibly deleting directories in the future Convert media processing backends to delete the queue directory (#254) Implement delete_dir in the FileStorage Conflicts: mediagoblin/media_types/image/processing.py mediagoblin/media_types/video/processing.py Conflicts, because those media_types already use the newer proc_state.delete_queue_file() method (which needs updating.
| * | | | | | | Removing docstring bit about delete_file possibly deleting directories in ↵Christopher Allan Webber2013-01-221-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the future I agree that delete_dir as a separate operation is a better way to do things, especially since there is a non-recursive deletion option that will politely fail if the directory is not empty.
| * | | | | | | Convert media processing backends to delete the queue directory (#254)Sebastian Spaeth2013-01-225-7/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We never deleted our queue directory which were created per submission. With the FileStorage backend being able to delete directories now, we can request the deletion of the task directory too. It will only be deleted if it is completely empty.
| * | | | | | | Implement delete_dir in the FileStorageSebastian Spaeth2013-01-222-3/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | plus options for deleting only empty directories and deleting them recursively. Not sure how cloudfile storage is or should be handled here. Are things such as a "directory" even a concept there?
* | | | | | | | Teach runtests.sh the -n arg.Elrond2013-04-061-0/+7
| |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If you give "-n 8" to runtests, it thinks, it got "-n" -- a nice option, ignored. "8" -- Oh! a single test named "8" to run, so no need to run all the GMG tests. Well, that's not what we want. So runtests now knows about -n taking an arg.
* | | | | | | Really removing nosetests things now! all assert_whatever removedChristopher Allan Webber2013-04-0611-96/+75
| | | | | | |
* | | | | | | Better docs for GenerateSlugMixin.Elrond2013-04-061-1/+9
| | | | | | |
* | | | | | | Use GenerateSlugMixin for collections.Elrond2013-04-063-14/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the new way of generating slugs also for collections. Also drop the dummy_db arg to check_collection_slug_used.
* | | | | | | Refactor generate_slug into a mixin.Elrond2013-04-061-9/+13
| |_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | Refactor the new generate_slug into a mixin to allow it to be used for collections also.
* | | | | | Merge branch '614_multi_test_config'Christopher Allan Webber2013-04-0418-229/+252
|\ \ \ \ \ \
| * | | | | | Switch test_app generation over to use py.test fixtures.Christopher Allan Webber2013-04-0418-251/+248
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By doing this, we can take advantage of py.test's ability to create temporary directories that are then cleaned up later during testing. This helps for sandboxing things. This also involves a ton of changes: - Changing the get_app stuff appropriately, getting rid of the setup_fresh_app decorator - Making said fixture - Switching over a billion tests to use it
| * | | | | | Oops, forgot to assign dump_old_app to self, heh.Christopher Allan Webber2013-04-041-0/+1
| | | | | | |
| * | | | | | Turning setup_fresh_app into SetupFreshApp, a decorator with options!Christopher Allan Webber2013-04-041-8/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This way people can pass in particular paste/mediagoblin configs that they want to use.
| * | | | | | Updating get_app to allow for passing in particular app configsChristopher Allan Webber2013-04-041-2/+17
| | | | | | |
* | | | | | | Also require pytest-xdist!Christopher Allan Webber2013-04-031-0/+1
|/ / / / / /
* | | | | | Use py.test --boxed for unit tests from now on!Christopher Allan Webber2013-04-032-11/+11
| | | | | |
* | | | | | These tests need to have a MediaGoblin app setup so they can connect to the db!Christopher Allan Webber2013-04-032-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | Thanks to py.test --boxed for helping discover that ;)
* | | | | | All mediagoblin tests now pass with py.test (switched setUp to setup)Christopher Allan Webber2013-04-038-10/+10
| | | | | |
* | | | | | Create ProcessingState.store_public.Elrond2013-04-031-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ideas is by Alon Levy. Use it in ProcessingState.copy_original for now.
* | | | | | Little refactoring for images: media_files.Elrond2013-04-031-8/+2
| | | | | | | | | | | | | | | | | | | | | | | | Set the media_files directly at the relvant places.
* | | | | | A bit of pep8: Remove useless whitespace.Elrond2013-03-301-5/+5
| | | | | |