Merge pull request #709 from openstax/fix-sso-for-localhost

Remove domain from sso cookie on dev and test
openstax · Sep 30, 2019 · 143f4fd · 143f4fd
2 parents 21e0a54 + 6444b30
commit 143f4fd
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/config/secrets.yml.example b/config/secrets.yml.example
@@ -80,7 +80,9 @@ development:
     cookie:
       name: <%= ENV["SSO_COOKIE_NAME"] || 'ox' %>
       options:
-        domain: <%= ENV["SSO_COOKIE_DOMAIN"] || "''" %>
+        <% if ENV.key?("SSO_COOKIE_DOMAIN") %>
+        domain: <%= ENV["SSO_COOKIE_DOMAIN"] %>
+        <% end %>
         secure: <%= ActiveModel::Type::Boolean.new.cast ENV.fetch(
                       "SSO_COOKIE_SECURE", false
                     ) %>
@@ -147,7 +149,6 @@ test:
     cookie:
       name: ox
       options:
-        domain: ''
         secure: false
         httponly: true