diff options
author | Eugen Rochko <eugen@zeonfederated.com> | 2016-10-06 21:33:33 +0200 |
---|---|---|
committer | Eugen Rochko <eugen@zeonfederated.com> | 2016-10-06 21:33:33 +0200 |
commit | d772db43441ebc28655b76f3662f02fa7fd48839 (patch) | |
tree | 1537747e9340dc0f13ea070f1a621e29c63a0973 /app | |
parent | 3554d638b31aa5ace2aa524db0165932c15d9a71 (diff) |
Fix #83 - if user cannot be found (or is self) throw error, don't return empty 200
This prevents the undefined profile from being opened in the first place on such an error
Diffstat (limited to 'app')
-rw-r--r-- | app/services/follow_service.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/services/follow_service.rb b/app/services/follow_service.rb index 4caf55078..2a9dc82c3 100644 --- a/app/services/follow_service.rb +++ b/app/services/follow_service.rb @@ -5,7 +5,7 @@ class FollowService < BaseService def call(source_account, uri) target_account = follow_remote_account_service.call(uri) - return nil if target_account.nil? || target_account.id == source_account.id + raise ActiveRecord::RecordNotFound if target_account.nil? || target_account.id == source_account.id follow = source_account.follow!(target_account) |