Use :message key, not :error for File::Service.

These bugs were introduced by myself at https://github.com/gitlabhq/gitlabhq/pull/7807

The fact that the refactoring lead to bugs means we should add more tests for those error cases =)

Also removed the `out[:error] = ''` which never happens: every place of the app first checks for `success`.