Skip to content

Commit

Permalink
magento/magento2 #8616:
Browse files Browse the repository at this point in the history
Unused mysql privileges required even if not used/not needed at all
- removed REFERENCES and EVENT privileges from DbValidator (also in Test Unit).
  • Loading branch information
Nino Aratari committed Mar 3, 2017
1 parent ae6a297 commit c268d58
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ public function testCheckDatabaseConnection()
['DELETE'],
['CREATE'],
['DROP'],
['REFERENCES'],
['INDEX'],
['ALTER'],
['CREATE TEMPORARY TABLES'],
Expand All @@ -65,7 +64,6 @@ public function testCheckDatabaseConnection()
['SHOW VIEW'],
['CREATE ROUTINE'],
['ALTER ROUTINE'],
['EVENT'],
['TRIGGER'],
];
$accessibleDbs = ['some_db', 'name', 'another_db'];
Expand Down
2 changes: 0 additions & 2 deletions setup/src/Magento/Setup/Validator/DbValidator.php
Original file line number Diff line number Diff line change
Expand Up @@ -146,7 +146,6 @@ private function checkDatabasePrivileges(\Magento\Framework\DB\Adapter\AdapterIn
'DELETE',
'CREATE',
'DROP',
'REFERENCES',
'INDEX',
'ALTER',
'CREATE TEMPORARY TABLES',
Expand All @@ -156,7 +155,6 @@ private function checkDatabasePrivileges(\Magento\Framework\DB\Adapter\AdapterIn
'SHOW VIEW',
'CREATE ROUTINE',
'ALTER ROUTINE',
'EVENT',
'TRIGGER'
];

Expand Down

0 comments on commit c268d58

Please sign in to comment.