Skip to content
  • Marcus Efraimsson's avatar
    53cd39fd
    Shouldn't be able to overwrite a dashboard if you don't have permissions (#10900) · 53cd39fd
    Marcus Efraimsson authored and Torkel Ödegaard's avatar Torkel Ödegaard committed
    * dashboards: new command for validating dashboard before update
    
    Removes validation logic from saveDashboard and later on use the new command for validating
    dashboard before saving a dashboard. This due to the fact that we need to validate permissions
    for overwriting other dashboards by uid and title.
    
    * dashboards: use the new command for validating dashboard before saving
    
    Had to refactor dashboard provisioning a bit to be able to sidetrack the permission validation
    in a somewhat reasonable way.
    Adds some initial tests of the dashboard repository, but needs to be extended later. At least
    now you can mock the dashboard guardian
    
    * dashboards: removes validation logic in the save dashboard api layer
    
    Use the dashboard repository solely for create/update dashboards and let it do all
    the validation. One exception regarding quota validation which still is in api layer
    since that logic is in a macaron middleware.
    Need to move out-commented api tests later.
    
    * dashboards: fix database tests for validate and saving dashboards
    
    * dashboards: rename dashboard repository to dashboard service
    
    Split the old dashboard repository interface in two new interfaces, IDashboardService and
    IDashboardProvisioningService. Makes it more explicit when using it from the provisioning package
    and there's no possibility of calling an incorrect method for saving a dashboard.
    
    * database: make the InitTestDB function available to use from other packages
    
    * dashboards: rename ValidateDashboardForUpdateCommand and some refactoring
    
    * dashboards: integration tests of dashboard service
    
    * dashboard: fix sqlstore test due to folder exist validation
    
    * dashboards: move dashboard service integration tests to sqlstore package
    
    Had to move it to the sqlstore package due to concurrency problems when running
    against mysql and postgres. Using InitTestDB from two packages added conflicts
    when clearing and running migrations on the test database
    
    * dashboards: refactor how to find id to be used for save permission check
    
    * dashboards: remove duplicated dashboard tests
    
    * dashboards: cleanup dashboard service integration tests
    
    * dashboards: handle save dashboard errors and return correct http status
    
    * fix: remove log statement
    
    * dashboards: import dashboard should use dashboard service
    
    Had to move alerting commands to models package due to problems with import cycles of packages.
    
    * dashboards: cleanup dashboard api tests and add some tests for post dashboard
    
    * dashboards: rename dashboard service interfaces
    
    * dashboards: rename dashboard guardian interface
    53cd39fd
    Shouldn't be able to overwrite a dashboard if you don't have permissions (#10900)
    Marcus Efraimsson authored and Torkel Ödegaard's avatar Torkel Ödegaard committed
    * dashboards: new command for validating dashboard before update
    
    Removes validation logic from saveDashboard and later on use the new command for validating
    dashboard before saving a dashboard. This due to the fact that we need to validate permissions
    for overwriting other dashboards by uid and title.
    
    * dashboards: use the new command for validating dashboard before saving
    
    Had to refactor dashboard provisioning a bit to be able to sidetrack the permission validation
    in a somewhat reasonable way.
    Adds some initial tests of the dashboard repository, but needs to be extended later. At least
    now you can mock the dashboard guardian
    
    * dashboards: removes validation logic in the save dashboard api layer
    
    Use the dashboard repository solely for create/update dashboards and let it do all
    the validation. One exception regarding quota validation which still is in api layer
    since that logic is in a macaron middleware.
    Need to move out-commented api tests later.
    
    * dashboards: fix database tests for validate and saving dashboards
    
    * dashboards: rename dashboard repository to dashboard service
    
    Split the old dashboard repository interface in two new interfaces, IDashboardService and
    IDashboardProvisioningService. Makes it more explicit when using it from the provisioning package
    and there's no possibility of calling an incorrect method for saving a dashboard.
    
    * database: make the InitTestDB function available to use from other packages
    
    * dashboards: rename ValidateDashboardForUpdateCommand and some refactoring
    
    * dashboards: integration tests of dashboard service
    
    * dashboard: fix sqlstore test due to folder exist validation
    
    * dashboards: move dashboard service integration tests to sqlstore package
    
    Had to move it to the sqlstore package due to concurrency problems when running
    against mysql and postgres. Using InitTestDB from two packages added conflicts
    when clearing and running migrations on the test database
    
    * dashboards: refactor how to find id to be used for save permission check
    
    * dashboards: remove duplicated dashboard tests
    
    * dashboards: cleanup dashboard service integration tests
    
    * dashboards: handle save dashboard errors and return correct http status
    
    * fix: remove log statement
    
    * dashboards: import dashboard should use dashboard service
    
    Had to move alerting commands to models package due to problems with import cycles of packages.
    
    * dashboards: cleanup dashboard api tests and add some tests for post dashboard
    
    * dashboards: rename dashboard service interfaces
    
    * dashboards: rename dashboard guardian interface
Loading