Remove code not longer needed (#2019)

Author: hectorcorrea

app/controllers/welcome_controller.rb

@@ -11,21 +11,5 @@ def index
     end
   end
 
-  def help
-    # Piggybacking on this page to pass custom HTTP headers to Mediaflux
-    # in a very controlled scenario.
-    root_ns = Rails.configuration.mediaflux["api_root_collection_namespace"]
-    parent_collection = Rails.configuration.mediaflux["api_root_collection_name"]
-    @test_path = Pathname.new(root_ns).join(parent_collection)
-    @test_http_headers = false
-    unless current_user.nil?
-      @test_http_headers = params["http-headers"] == "true"
-      request = if @test_http_headers
-                  Mediaflux::AssetExistRequest.new(session_token: current_user.mediaflux_session, path: @test_path, session_user: current_user)
-                else
-                  Mediaflux::AssetExistRequest.new(session_token: current_user.mediaflux_session, path: @test_path)
-                end
-      @test_path_exist = request.exist?
-    end
-  end
+  def help; end
 end

app/models/mediaflux/actor_self_describe_request.rb

@@ -3,8 +3,8 @@ module Mediaflux
   class ActorSelfDescribeRequest < Request
     # Constructor
     # @param session_token [String] the API token for the authenticated session
-    def initialize(session_token:, session_user: nil)
-      super(session_token: session_token, session_user: session_user)
+    def initialize(session_token:)
+      super(session_token: session_token)
     end
 
     # Specifies the Mediaflux service to use

app/models/mediaflux/asset_exist_request.rb

@@ -3,8 +3,8 @@ module Mediaflux
   class AssetExistRequest < Request
     # Constructor
     # @param session_token [String] the API token for the authenticated session
-    def initialize(session_token:, path:, session_user: nil)
-      super(session_token: session_token, session_user: session_user)
+    def initialize(session_token:, path:)
+      super(session_token: session_token)
       @path = path
     end
 

app/models/mediaflux/request.rb

@@ -47,11 +47,10 @@ def self.find_or_create_http_client
       # @param file [File] any upload file required for the POST request
       # @param session_token [String] the API token for the authenticated session
       # @param http_client [Net::HTTP::Persistent] HTTP client for transmitting requests to the Mediaflux server API
-      def initialize(file: nil, session_token: nil, http_client: nil, session_user: nil)
+      def initialize(file: nil, session_token: nil, http_client: nil)
         @http_client = http_client || self.class.find_or_create_http_client
         @file = file
         @session_token = session_token
-        @session_user = session_user
       end
 
       # Resolves the HTTP request against the Mediaflux API
@@ -154,8 +153,6 @@ def build_http_request(name:, form_file: nil)
           request = self.class.build_post_request
 
           log_xml_request(xml_payload)
-          set_authentication_headers(request)
-
           if form_file.nil?
             request["Content-Type"] = "text/xml; charset=utf-8"
             request.body = xml_payload(name:)
@@ -181,13 +178,5 @@ def log_xml_request(xml_payload)
           Rails.logger.debug(xml_payload.gsub(password_element.to_s, "<password>***</password>"))
         end
       end
-
-      # Authentication code to push a few custom HTTP headers to Mediaflux
-      # Eventually the `session_user` will need to be an object that provides the timeout value.
-      def set_authentication_headers(request)
-        return if @session_user.nil?
-
-        request["mediaflux.sso.user"] = @session_user.uid
-      end
     end
 end

app/models/mediaflux/script_file_list_init_request.rb

@@ -4,8 +4,8 @@ class ScriptFileListInitRequest < Request
     # Constructor
     # @param session_token [String] the API token for the authenticated session
     # @param path [String] path to the collection we want the list of files for
-    def initialize(session_token:, path:, session_user: nil)
-      super(session_token: session_token, session_user: session_user)
+    def initialize(session_token:, path:)
+      super(session_token: session_token)
       @path = path
     end
 

app/models/mediaflux/script_file_list_iterate_request.rb

@@ -4,8 +4,8 @@ class ScriptFileListIterateRequest < Request
     # Constructor
     # @param session_token [String] the API token for the authenticated session
     # @param iterator [Int] the iterator returned by ScriptFileListInitRequest
-    def initialize(session_token:, iterator:, session_user: nil)
-      super(session_token: session_token, session_user: session_user)
+    def initialize(session_token:, iterator:)
+      super(session_token: session_token)
       @iterator = iterator
     end
 

app/models/mediaflux/script_make_executable_request.rb

@@ -4,8 +4,8 @@ class ScriptMakeExecutableRequest < Request
     # Constructor
     # @param session_token [String] the API token for the authenticated session
     # @param path [String] full path to the script to mark as executable
-    def initialize(session_token:, path:, session_user: nil)
-      super(session_token: session_token, session_user: session_user)
+    def initialize(session_token:, path:)
+      super(session_token: session_token)
       @path = path
     end
 

app/models/mediaflux/script_upload_request.rb

@@ -6,8 +6,8 @@ class ScriptUploadRequest < Request
     # @param namespace [String] namespace where the script will be saved
     # @param name [String] name of the script
     # @param url [String] URL where the content of the script will be fetched from
-    def initialize(session_token:, namespace:, name:, url:, session_user: nil)
-      super(session_token: session_token, session_user: session_user)
+    def initialize(session_token:, namespace:, name:, url:)
+      super(session_token: session_token)
       @namespace = namespace
       @name = name
       @url = url

app/views/welcome/help.html.erb

@@ -1,11 +1,3 @@
 <p>TigerData is still under construction, and you are not registered as a tester at this time.</p>
 
 <p>If you have any questions, please contact the team at <%= link_to("[email protected]", "mailto:[email protected]") %>.</p>
-
-<% if current_user %>
-  <h3>HTTP headers test</h3>
-  <p>Custom HTTP headers: <%= @test_http_headers %> </p>
-  <p>User: <%= current_user.uid %> </p>
-  <p>Path: <%= @test_path %></p>
-  <p>Exist?: <%= @test_path_exist %></p>
-<% end %>