Skip to content
Snippets Groups Projects
Unverified Commit 90039279 authored by bryan's avatar bryan Committed by GitHub
Browse files

Adds a property to a defsetting (#44990)

* mark a defsetting as being :sensitive?

* add test
parent 8786c23a
No related branches found
No related tags found
No related merge requests found
......@@ -262,6 +262,7 @@
(defsetting premium-embedding-token ; TODO - rename this to premium-features-token?
(deferred-tru "Token for premium features. Go to the MetaStore to get yours!")
:audit :never
:sensitive? true
:setter
(fn [new-value]
;; validate the new value if we're not unsetting it
......
......@@ -469,6 +469,18 @@
(-> (mt/client :get 200 "session/properties" {:request-options {:headers {"x-metabase-locale" "es"}}})
:engines :h2 :details-fields first :display-name)))))))
(deftest properties-skip-sensitive-test
(reset-throttlers!)
(testing "GET /session/properties"
(testing "don't return the token for admins"
(is (= nil
(-> (mt/client :get 200 "session/properties" (mt/user->credentials :crowberto))
keys #{:premium-embedding-token}))))
(testing "don't return the token for non-admins"
(is (= nil
(-> (mt/client :get 200 "session/properties" (mt/user->credentials :rasta))
keys #{:premium-embedding-token}))))))
;;; ------------------------------------------- TESTS FOR GOOGLE SIGN-IN ---------------------------------------------
(deftest google-auth-test
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment