[Actions] disallow is_write outside of actions api (#24712)
* disallow writable queries outside of actions api - handles cards, dashboards, embedding and pulses - tests for disallowing writable query execution - fix namespace decl * fix ns decl * use `false?` to avoid any nil cards causing a 405 * check that the card is there * Revert "check that the card is there" This reverts commit 5b56e2d00291bd21c0461a0567537769b5f73a83. * Revert "use `false?` to avoid any nil cards causing a 405" This reverts commit 72569d353ee5830f37d1a14c46289afbe511841b.
Showing
- src/metabase/api/common.clj 7 additions, 0 deletionssrc/metabase/api/common.clj
- src/metabase/api/embed.clj 1 addition, 1 deletionsrc/metabase/api/embed.clj
- src/metabase/api/public.clj 4 additions, 2 deletionssrc/metabase/api/public.clj
- src/metabase/pulse.clj 2 additions, 0 deletionssrc/metabase/pulse.clj
- src/metabase/query_processor/card.clj 2 additions, 1 deletionsrc/metabase/query_processor/card.clj
- src/metabase/query_processor/dashboard.clj 1 addition, 0 deletionssrc/metabase/query_processor/dashboard.clj
- test/metabase/api/card_test.clj 36 additions, 6 deletionstest/metabase/api/card_test.clj
- test/metabase/api/dashboard_test.clj 82 additions, 55 deletionstest/metabase/api/dashboard_test.clj
- test/metabase/api/embed_test.clj 2 additions, 2 deletionstest/metabase/api/embed_test.clj
- test/metabase/api/preview_embed_test.clj 21 additions, 3 deletionstest/metabase/api/preview_embed_test.clj
- test/metabase/api/public_test.clj 18 additions, 1 deletiontest/metabase/api/public_test.clj
Loading
Please register or sign in to comment