Merge pull request #441 from Sgoettschkes/issue/440

Not reading dsn config at compile time

Author: mitchellhenke

lib/sentry/config.ex

@@ -79,7 +79,7 @@ defmodule Sentry.Config do
 
   @deprecated "Use root_source_code_paths/0 instead"
   def root_source_code_path do
-    path = get_config(:root_source_code_path)
+    path = get_config(:root_source_code_path, check_dsn: false)
 
     if path do
       path
@@ -97,8 +97,8 @@ defmodule Sentry.Config do
   # We should deprecate this the :root_source_code_path completely in the next major
   # release.
   def root_source_code_paths do
-    paths = get_config(:root_source_code_paths)
-    path = get_config(:root_source_code_path)
+    paths = get_config(:root_source_code_paths, check_dsn: false)
+    path = get_config(:root_source_code_path, check_dsn: false)
 
     cond do
       not is_nil(path) and not is_nil(paths) ->

test/config_test.exs

@@ -113,5 +113,11 @@ defmodule Sentry.ConfigTest do
 
       assert Config.root_source_code_paths() == ["/test"]
     end
+
+    test "call to :root_source_code_paths does not read dsn env" do
+      modify_env(:sentry, dsn: {:system, "DSN", required: true})
+
+      Config.root_source_code_paths()
+    end
   end
 end