Merge pull request #922 from jotolo/improvements/insecure-url-comments

improvements/replacing http with https on comments
appsignal · Feb 23, 2023 · 2ceeddb · 2ceeddb
2 parents a117d50 + 865ffb1
commit 2ceeddb
Show file tree

Hide file tree

Showing 8 changed files with 26 additions and 26 deletions.
diff --git a/lib/appsignal.rb b/lib/appsignal.rb
@@ -74,8 +74,8 @@ def testing?
     # This is not required for the automatic integrations AppSignal offers, but
     # this is required for all non-automatic integrations and pure Ruby
     # applications. For more information, see our [integrations
-    # list](http://docs.appsignal.com/ruby/integrations/) and our [Integrating
-    # AppSignal](http://docs.appsignal.com/ruby/instrumentation/integrating-appsignal.html)
+    # list](https://docs.appsignal.com/ruby/integrations/) and our [Integrating
+    # AppSignal](https://docs.appsignal.com/ruby/instrumentation/integrating-appsignal.html)
     # guide.
     #
     # To start the logger see {.start_logger}.

diff --git a/lib/appsignal/cli/demo.rb b/lib/appsignal/cli/demo.rb
@@ -35,9 +35,9 @@ class CLI
     # @since 2.0.0
     # @see Appsignal::Demo
     # @see Appsignal::CLI::Install
-    # @see http://docs.appsignal.com/ruby/command-line/demo.html
+    # @see https://docs.appsignal.com/ruby/command-line/demo.html
     #   AppSignal demo documentation
-    # @see http://docs.appsignal.com/support/debugging.html
+    # @see https://docs.appsignal.com/support/debugging.html
     #   Debugging AppSignal guide
     # @api private
     class Demo

diff --git a/lib/appsignal/cli/install.rb b/lib/appsignal/cli/install.rb
@@ -126,7 +126,7 @@ def install_for_sinatra(config)
           puts "  require 'appsignal/integrations/sinatra'"
           puts
           puts "  You can find more information in the documentation:"
-          puts "  http://docs.appsignal.com/ruby/integrations/sinatra.html"
+          puts "  https://docs.appsignal.com/ruby/integrations/sinatra.html"
           press_any_key
           done_notice
         end
@@ -144,7 +144,7 @@ def install_for_padrino(config)
           puts "  require 'appsignal/integrations/padrino"
           puts
           puts "  You can find more information in the documentation:"
-          puts "  http://docs.appsignal.com/ruby/integrations/padrino.html"
+          puts "  https://docs.appsignal.com/ruby/integrations/padrino.html"
           press_any_key
           done_notice
         end
@@ -159,7 +159,7 @@ def install_for_grape(config)
 
           puts "Manual Grape configuration needed"
           puts "  See the installation instructions at:"
-          puts "  http://docs.appsignal.com/ruby/integrations/grape.html"
+          puts "  https://docs.appsignal.com/ruby/integrations/grape.html"
           press_any_key
           done_notice
         end
@@ -177,7 +177,7 @@ def install_for_hanami(config)
           puts "  require 'appsignal/integrations/hanami'"
           puts
           puts "  You can find more information in the documentation:"
-          puts "  http://docs.appsignal.com/ruby/integrations/hanami.html"
+          puts "  https://docs.appsignal.com/ruby/integrations/hanami.html"
           press_any_key
           done_notice
         end
@@ -234,7 +234,7 @@ def configure(config, environments, name_overwritten)
               end
               puts
               puts "  See the documentation for more configuration options:"
-              puts "  http://docs.appsignal.com/gem-settings/configuration.html"
+              puts "  https://docs.appsignal.com/gem-settings/configuration.html"
               press_any_key
               break
             end

diff --git a/lib/appsignal/helpers/instrumentation.rb b/lib/appsignal/helpers/instrumentation.rb
@@ -197,9 +197,9 @@ def listen_for_error(
       #   used to send the error.
       # @return [void]
       #
-      # @see http://docs.appsignal.com/ruby/instrumentation/exception-handling.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/exception-handling.html
       #   Exception handling guide
-      # @see http://docs.appsignal.com/ruby/instrumentation/tagging.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/tagging.html
       #   Tagging guide
       # @since 0.6.0
       def send_error(
@@ -296,7 +296,7 @@ def send_error(
       # @return [void]
       #
       # @see Transaction#set_error
-      # @see http://docs.appsignal.com/ruby/instrumentation/exception-handling.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/exception-handling.html
       #   Exception handling guide
       # @since 0.6.6
       def set_error(exception, tags = nil, namespace = nil)
@@ -435,7 +435,7 @@ def set_namespace(namespace)
       # @return [void]
       #
       # @see Transaction.set_tags
-      # @see http://docs.appsignal.com/ruby/instrumentation/tagging.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/tagging.html
       #   Tagging guide
       def tag_request(tags = {})
         return unless active?
@@ -468,7 +468,7 @@ def tag_request(tags = {})
       # @return [void]
       #
       # @see Transaction#add_breadcrumb
-      # @see http://docs.appsignal.com/ruby/instrumentation/breadcrumbs.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/breadcrumbs.html
       #   Breadcrumb reference
       # @since 2.12.0
       def add_breadcrumb(category, action, message = "", metadata = {}, time = Time.now.utc)
@@ -513,9 +513,9 @@ def add_breadcrumb(category, action, message = "", metadata = {}, time = Time.no
       #
       # @see Appsignal::Transaction#instrument
       # @see .instrument_sql
-      # @see http://docs.appsignal.com/ruby/instrumentation/instrumentation.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/instrumentation.html
       #   AppSignal custom instrumentation guide
-      # @see http://docs.appsignal.com/api/event-names.html
+      # @see https://docs.appsignal.com/api/event-names.html
       #   AppSignal event naming guide
       # @since 1.3.0
       def instrument(
@@ -557,9 +557,9 @@ def instrument(
       # @return [Object] Returns the block's return value.
       #
       # @see .instrument
-      # @see http://docs.appsignal.com/ruby/instrumentation/instrumentation.html
+      # @see https://docs.appsignal.com/ruby/instrumentation/instrumentation.html
       #   AppSignal custom instrumentation guide
-      # @see http://docs.appsignal.com/api/event-names.html
+      # @see https://docs.appsignal.com/api/event-names.html
       #   AppSignal event naming guide
       # @since 2.0.0
       def instrument_sql(name, title = nil, body = nil, &block)

diff --git a/lib/appsignal/marker.rb b/lib/appsignal/marker.rb
@@ -18,7 +18,7 @@ module Appsignal
   #     Set config does not override data set in {#marker_data}.
   #
   # @see Appsignal::CLI::NotifyOfDeploy
-  # @see http://docs.appsignal.com/appsignal/terminology.html#markers
+  # @see https://docs.appsignal.com/appsignal/terminology.html#markers
   #   Terminology: Deploy marker
   # @api private
   class Marker

diff --git a/lib/appsignal/transaction.rb b/lib/appsignal/transaction.rb
@@ -159,7 +159,7 @@ def params
     # @return [void]
     #
     # @see Appsignal.tag_request
-    # @see http://docs.appsignal.com/ruby/instrumentation/tagging.html
+    # @see https://docs.appsignal.com/ruby/instrumentation/tagging.html
     #   Tagging guide
     def set_tags(given_tags = {})
       @tags.merge!(given_tags)
@@ -177,7 +177,7 @@ def set_tags(given_tags = {})
     # @return [void]
     #
     # @see Appsignal.add_breadcrumb
-    # @see http://docs.appsignal.com/ruby/instrumentation/breadcrumbs.html
+    # @see https://docs.appsignal.com/ruby/instrumentation/breadcrumbs.html
     #   Breadcrumb reference
     def add_breadcrumb(category, action, message = "", metadata = {}, time = Time.now.utc)
       @breadcrumbs.push(

diff --git a/resources/appsignal.yml.erb b/resources/appsignal.yml.erb
@@ -26,7 +26,7 @@ default: &defaults
   #   - SystemExit
   #   - SystemStackError
 
-  # See http://docs.appsignal.com/ruby/configuration/options.html for
+  # See https://docs.appsignal.com/ruby/configuration/options.html for
   # all configuration options.
 
 # Configuration per environment, leave out an environment or set active

diff --git a/spec/lib/appsignal/cli/install_spec.rb b/spec/lib/appsignal/cli/install_spec.rb
@@ -469,7 +469,7 @@ def run
             "Installing for Sinatra",
             "Sinatra requires some manual configuration.",
             "require 'appsignal/integrations/sinatra'",
-            "http://docs.appsignal.com/ruby/integrations/sinatra.html"
+            "https://docs.appsignal.com/ruby/integrations/sinatra.html"
           ]
         end
         let(:app_name) { "Test app" }
@@ -536,7 +536,7 @@ def run
           [
             "Installing for Padrino",
             "Padrino requires some manual configuration.",
-            "http://docs.appsignal.com/ruby/integrations/padrino.html"
+            "https://docs.appsignal.com/ruby/integrations/padrino.html"
           ]
         end
         let(:app_name) { "Test app" }
@@ -603,7 +603,7 @@ def run
           [
             "Installing for Grape",
             "Manual Grape configuration needed",
-            "http://docs.appsignal.com/ruby/integrations/grape.html"
+            "https://docs.appsignal.com/ruby/integrations/grape.html"
           ]
         end
         let(:app_name) { "Test app" }
@@ -670,7 +670,7 @@ def run
           [
             "Installing for Hanami",
             "Hanami requires some manual configuration.",
-            "http://docs.appsignal.com/ruby/integrations/hanami.html"
+            "https://docs.appsignal.com/ruby/integrations/hanami.html"
           ]
         end
         let(:app_name) { "Test app" }