5018: Add permission check for database altering before allowing user…

… to install SQL Server node with log-based replication (#159) * 5018: add permission check for altering databases to wizard for sql server log-based repl * 5018: add permission check for altering databases to wizard for sql server log-based repl
JumpMind · Jun 10, 2021 · 2c4845d · Christian-PK · Sep 21, 2021 · rudiejd
1 parent 8fc6797
commit 2c4845d
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,7 +1,7 @@
 /.DS_Store
 /.project
 *.iml
-
+.clover/*
 /.idea/*
 *.ipr
 *.iws
diff --git a/symmetric-jdbc/src/main/java/org/jumpmind/db/platform/mssql/MsSql2008DatabasePlatform.java b/symmetric-jdbc/src/main/java/org/jumpmind/db/platform/mssql/MsSql2008DatabasePlatform.java
@@ -72,8 +72,22 @@ public long getEstimatedRowCount(Table table) {
     @Override
     protected PermissionResult getLogMinePermission() {
         final PermissionResult result = new PermissionResult(PermissionType.LOG_MINE, "");
-        result.setStatus(Status.PASS);
+        try {
+            if (getSqlTemplate().queryForInt("SELECT COUNT(*) FROM fn_my_permissions(NULL, 'SERVER') WHERE permission_name='ALTER'") > 0) {
+                result.setStatus(Status.PASS);
+            } else {
+                result.setStatus(Status.FAIL);
+                result.setSolution("Grant alter any database to this user."); 
+            }
+
+        } catch (Exception e) {
+            result.setSolution("Error occurred checking user permissions");
+            result.setException(e);
+        }
         return result;
     }
 
+
+
+
 }