about summary refs log tree commit diff
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2016-12-29 17:23:27 +0100
committerEugen Rochko <eugen@zeonfederated.com>2016-12-29 17:23:27 +0100
commit8724094ed0e531f4435bf2784c9c1b7176acd764 (patch)
treef40d8c65f178c89030fdf9ec3351707b4723e712
parentd7dc84439c60069a0cb9eeca81dc61c297a8667b (diff)
Support remote follow request providing URL instead of acct
-rw-r--r--app/assets/stylesheets/accounts.scss4
-rw-r--r--app/controllers/authorize_follow_controller.rb19
-rw-r--r--app/views/authorize_follow/new.html.haml3
3 files changed, 24 insertions, 2 deletions
diff --git a/app/assets/stylesheets/accounts.scss b/app/assets/stylesheets/accounts.scss
index 5d0963307..748bb8224 100644
--- a/app/assets/stylesheets/accounts.scss
+++ b/app/assets/stylesheets/accounts.scss
@@ -337,6 +337,10 @@
     overflow: hidden;
     margin-bottom: 15px;
 
+    &:last-child {
+      margin-bottom: 0;
+    }
+
     & > div {
       float: left;
       margin-right: 10px;
diff --git a/app/controllers/authorize_follow_controller.rb b/app/controllers/authorize_follow_controller.rb
index a276250a4..ca72c9691 100644
--- a/app/controllers/authorize_follow_controller.rb
+++ b/app/controllers/authorize_follow_controller.rb
@@ -6,7 +6,14 @@ class AuthorizeFollowController < ApplicationController
   before_action :authenticate_user!
 
   def new
-    @account = FollowRemoteAccountService.new.call(params[:acct])
+    uri = Addressable::URI.parse(params[:acct])
+
+    if uri.path && %w(http https).include?(uri.scheme)
+      set_account_from_url
+    else
+      set_account_from_acct
+    end
+
     render :error if @account.nil?
   end
 
@@ -21,4 +28,14 @@ class AuthorizeFollowController < ApplicationController
   rescue ActiveRecord::RecordNotFound, Mastodon::NotPermitted
     render :error
   end
+
+  private
+
+  def set_account_from_url
+    @account = FetchRemoteAccountService.new.call(params[:acct])
+  end
+
+  def set_account_from_acct
+    @account = FollowRemoteAccountService.new.call(params[:acct])
+  end
 end
diff --git a/app/views/authorize_follow/new.html.haml b/app/views/authorize_follow/new.html.haml
index 7368b834a..44bf575ff 100644
--- a/app/views/authorize_follow/new.html.haml
+++ b/app/views/authorize_follow/new.html.haml
@@ -14,7 +14,8 @@
           %strong= display_name(@account)
           %span= "@#{@account.acct}"
 
-      .account__header__content= Formatter.instance.simplified_format(@account)
+      - unless @account.note.blank?
+        .account__header__content= Formatter.instance.simplified_format(@account)
 
   = form_tag authorize_follow_path, method: :post, class: 'simple_form' do
     = hidden_field_tag :acct, @account.acct