Merge remote-tracking branch 'remotes/nyergler/issue-653-csrf-exception'

Issue 653: This time for sure!

Issue 653: Handle the case where request.vary is None

Issue 653: Don't throw exception if response has no vary header.

Mark two strings for translation

Final step for non-force-conversion to jpeg

Don't force-convert resized images to JPEG.

That's just not nice for those of us who like transparency!

Probably should have MANIFEST.in checked in, for doing python sdists

640: Configuration files should mention their _local versions

Thanks go to Aleksej Serdjukov for bringing this up and
providing the patch in the bug!

Merge branch 'idea/check_mgoblin_static'

* idea/check_mgoblin_static:
  Move TestingMiddleware to tests/tools.py
  Some docs for the TestingMiddleware
  591: Let Testing Middleware check for staticdirect() usage

Move TestingMiddleware to tests/tools.py

This middleware isn't needed outside of the tests, so let's
just put it there.

Some docs for the TestingMiddleware

To make the TestingMiddleware actually more useful in the
future, start to document it.

Merge branch 'master' of gitorious.org:mediagoblin/mediagoblin

Updating translations

Tiny padding change to vertically center navigation button arrows

Small style changes to navigation buttons

On second thought, let's use this title for forgot_password.html

Text edits and rearrangements to login.html

Add translation brackety things to forgot-password page, I forgot those

Text changes to forgot-paddword page

Text changes to front page, as described in ticket #637

Several small changes (reordering/restyling) for media descriptions and comments

Correcting the mediagoblin static/media aliases in the deployment guide

A correct but compact set of instructions for fastcgi on nginx

Indentation was off by one ;)

We left out a critical fastcgi variable from the nginx config

Recommendation to install flup!

0.1.0, also in the docs file! ;)

0.1.0!

Removed the bloof print debugging

Make sure you can't force comments on entries that don't exist.

See: http://bugs.foocorp.net/issues/621

Updated logo from Thorsten Wilms!

Updated translations

Merge remote branch 'remotes/j1mc/master'

Added configuration to the index

A cop-out section for all other config changes

Enabling email notifications documentation!

docs - tweaked configuration.rst for style.

Making docs changes per Jim's suggestions.

Added the "making local copies" section to configuration.rst

First section of configuring mediagoblin

Configuring MediaGoblin, a loose sketch of documentation.

docs: editing/tweaking deployment documentation

Added notes on a more permanent mediagoblin process.

Make sure we run fcgi in forked mode... mediagoblin is not currently thread safe!

No reason to include commented-out things.

Recommend that users use a non-privelaged user (thanks Elrond for the text)

Let's comply with the Filesystem Hierarchy Standard

... and make elrond and tychoish happy :)

If I"m telling people to use fastcgi, we should probably use fastcgi ;)

Hook mediagoblin up to nginx

More easy options for deploying without having to modify paste.ini...

Added a lot more details to deploying.rst

Added a "broadcast" server config and a bit of helpful commenting (I hope!)

[620] Allow for multiple mediagoblin configs to be set in config.ini, use first

This way we can copy paste.ini into paste_local.ini but not have to
update it to find mediagoblin_local.ini.

Default to mediagoblin_local.ini if available in ./bin/gmg commands

Merge remote branch 'remotes/elrond/misc/lazyserver_paste_local'

Small style changes: decrease button text size, change navigation button background color

Removing this extra padding after form_field_box.

Maybe I'm wrong to do this, but it does seem excessive to me.

I should talk to Jef about it probably :)

Removing comment_form link which doesn't do anything?

Unclosed anchor tags in the tags!  Fixed :)

Only show the "post a comment" link if there's a logged in user

Renamed "Temporary button holder" to "actions"

Removed suggestion that the user run ./bin/buildout

Added an extra warning about wipealldata being pretty inflexible.

lazyserver.sh: Try paste_local.ini before paste.ini

If users have a paste_local.ini, use that one instead of
the shipped paste.ini.
That way local changes will not interfere with the main
shipped config.

Also improve the help string (./lazyserver.sh -h) a bit.

flup, not flerp!

Merge remote branch 'remotes/elrond/misc/add_db_field'

Merge remote branch 'remotes/elrond/refactor/gmg_cf_option'

Add a flup server option to the default paste.ini to make deployment a bit easier.

Should be Lato-Regular.ttf not Lato-Regular.woff (why was it even working?)

Make gmg's -cf option a global option

All gmg's subcommands used to have a -cf option to set the
config file. Instead make this a gmg global option.
This means:
  bin/gmg migrate -cf mediagoblin_2.ini
gets
  bin/gmg -cf mediagoblin_2.ini migrate

Refactor gmg's cf option into a function

Many (all?) gmg subcommands take a -cf option to change the
used config file. This options used to be created in each
subcommand's parse_setup. Add a helper function and use it
around.

Change unit tests to use add_table_field

One of the test migrations adds a field. So rewrite it to
use the new add_table_field helper function.  This is a
good test for it.

Factor out a add_table_field function

Migrations often just add a new field to some
table/collection. So just have a nice helper function for
this!

591: Let Testing Middleware check for staticdirect() usage

All references to static, shipped content should go via
request.staticdirect().

So insert a TestingMiddleware in the chain, which tests for
this on all requests/reponses happening in the test suite.
It's a simple text search for a bogus usage pattern.

Require an older version of WebOb as a stopgap fix for Issue #622

Hopefully they'll release a new version with my patch soon!

Now that we're encouraging in-package virtualenv use, we should update
.gitignore.  Done! ;)

Hotfix: Renaming cookie broke unit tests

The name of the cookie was hardcoded in the unit tests and
while renaming the cookie this part was missed.

So now read the cookie_name from the config in the test.
This also tests for the config item existing!

Support for older webobs and newer webobs both in accept language mess :)

Fixes issue662, MediaGoblin was broken by recent WebOb release.

From WebOb release notes:

  "Accept.best_matches() is gone; use list(request.accept) or
  request.accept.best_match(..) instead (applies to all Accept-*
  headers) or similar with request.accept_language."

... seems that's what borked it...

Merge remote branch 'remotes/elrond/misc/i592_use_full_path'

Merge remote branch 'remotes/elrond/idea/csrf_improvement'

Merge remote branch 'aaronw/bug601_blank_comments'

* aaronw/bug601_blank_comments:
  Make Comment posted! translatable.
  Update english translation file.
  Added a check to prevent blank messages from being posted.

Conflicts:
mediagoblin/i18n/en/LC_MESSAGES/mediagoblin.po

Merge remote branch 'aaronw/bug608_checkbox_labels'

* aaronw/bug608_checkbox_labels:
  No need to translate label name.
  Add label tag to fix translation issue.
  Update english translation file.
  Checkbox label is being translated in the form definition -- translating it in the template is redundant and breaks <label> functionality.

Conflicts:
mediagoblin/i18n/en/LC_MESSAGES/mediagoblin.po

Improve indentation a bit

Only change: Indentation, nothing else!

Merge remote branch 'aaronw/feature469_comment_form_under_comments'

* aaronw/feature469_comment_form_under_comments:
  Put the trans tag in the right place.
  Move comment form below comments.

Put the trans tag in the right place.

Some mostly cosmetic changes to CSRF

* remove max_age - A session cookie is better, because it's
  a session thing, really.
* Call the cookie mediagoblin_csrftoken, much clearer.
* Use the SCRIPT_NAME for the path of the cookie, so that
  the cookie is sent back to the right place only.
  Alternatively the path= parameter could be removed, so
  that it defaults to '/'.
* call the randomness function only once, instead of twice.
  64 bits should be enough. If really more bits are needed,
  increase the number.
* Just give the number as cookie. No point in md5 and
  hexdigest in my view (those functions just make another
  representation).
* getrandbits gets a bit count directly, simpler API

Overhauls docs

* Removes a bunch of content that doesn't need to be in the suer manual
  anymore.
* Fixes issues so it's more readable in source form.
* Adds help chapter.
* Moves links out of paragraphs to reduce line length.
* Cleans up some language.
* Fixes some links.

Fixing 404s, related to recent util refactoring.

Make Comment posted! translatable.

No need to translate label name.

Add label tag to fix translation issue.

Move comment form below comments.

Update english translation file.

Update english translation file.

Added a check to prevent blank messages from being posted.

Checkbox label is being translated in the form definition -- translating it in the template is redundant and breaks <label> functionality.

i592: Use full path in various places

When running mediagoblin in a sub path on a web server,
most things inside mediagoblin need the "inside path", but
when generating URLs for the webbrowser, full paths are
needed.

urlgen and routes already do that.

Some (mostly pagination and login) need the URL of the
current page. They used request.path_info. But this is the
"inside" path, not the full.

So now there is request.full_path and its used in various
places.

Adding csrf token fields to the forgot password calls