From af36089fe055866c35941c0d0fd5dace380b7092 Mon Sep 17 00:00:00 2001
From: William Enright <william.enright@savoirfairelinux.com>
Date: Fri, 22 May 2020 15:29:03 -0400
Subject: [PATCH] removed redundant methods from authentication module,
 refactored checkVersion method for manifest file checking

---
 .../authmodule/UserAuthenticationModule.java  | 14 +++++++
 .../authmodule/AuthenticationModule.java      |  4 --
 .../jami/jams/common/utils/LicenseUtils.java  | 40 +------------------
 3 files changed, 16 insertions(+), 42 deletions(-)

diff --git a/authentication-module/src/main/java/net/jami/jams/authmodule/UserAuthenticationModule.java b/authentication-module/src/main/java/net/jami/jams/authmodule/UserAuthenticationModule.java
index b1c4d7c0..a5e58e67 100644
--- a/authentication-module/src/main/java/net/jami/jams/authmodule/UserAuthenticationModule.java
+++ b/authentication-module/src/main/java/net/jami/jams/authmodule/UserAuthenticationModule.java
@@ -172,4 +172,18 @@ public class UserAuthenticationModule implements AuthenticationModule {
         return (RSAPublicKey) publicKey;
     }
 
+    @Override
+    public char[] getOTP(String username) {
+
+        if(datastore.userExists(username)){
+            StatementList statementList = new StatementList();
+            StatementElement statementElement = new StatementElement("username","=",username,"");
+            statementList.addStatement(statementElement);
+            User user = datastore.getUserDao().getObjects(statementList).get(0);
+            return (user.getPassword()).toCharArray();
+        }
+
+        return new char[0];
+    }
+
 }
diff --git a/jams-common/src/main/java/net/jami/jams/common/authmodule/AuthenticationModule.java b/jams-common/src/main/java/net/jami/jams/common/authmodule/AuthenticationModule.java
index bfa65444..a32fe5f6 100644
--- a/jams-common/src/main/java/net/jami/jams/common/authmodule/AuthenticationModule.java
+++ b/jams-common/src/main/java/net/jami/jams/common/authmodule/AuthenticationModule.java
@@ -20,9 +20,5 @@ public interface AuthenticationModule {
     boolean testModuleConfiguration(AuthenticationSourceType type, String configuration);
     boolean createUser(AuthenticationSourceType type, String realm, NameServer nameServer, User user);
     RSAPublicKey getAuthModulePubKey();
-    boolean updateReset(User user, int needsReset);
-    boolean userExists(User user);
-    boolean userNeedsReset(User user);
-    boolean setNewOneTimePassword(String username, String otp);
     char[] getOTP(String username);
 }
diff --git a/jams-common/src/main/java/net/jami/jams/common/utils/LicenseUtils.java b/jams-common/src/main/java/net/jami/jams/common/utils/LicenseUtils.java
index 6dda71e7..5d55a05b 100644
--- a/jams-common/src/main/java/net/jami/jams/common/utils/LicenseUtils.java
+++ b/jams-common/src/main/java/net/jami/jams/common/utils/LicenseUtils.java
@@ -21,45 +21,9 @@ public class LicenseUtils {
         String resp = "";
         File file = new File(jarPath);
         JarFile jar = new JarFile(file);
-
-        // first get all directories,
-        // then make those directory on the destination Path
-        for (Enumeration<JarEntry> enums = jar.entries(); enums.hasMoreElements(); ) {
-            JarEntry entry = (JarEntry) enums.nextElement();
-
-            String fileName = destinationDir + File.separator + entry.getName();
-            File f = new File(fileName);
-
-            if (fileName.endsWith("/")) {
-                f.mkdirs();
-            }
-        }
-
-        //now create all files
-        for (Enumeration<JarEntry> enums = jar.entries(); enums.hasMoreElements(); ) {
-            JarEntry entry = (JarEntry) enums.nextElement();
-            if (entry.toString().contains("META-INF/MANIFEST.MF")) {
-                String fileName = destinationDir + File.separator + entry.getName();
-                File f = new File(fileName);
-
-                if (!fileName.endsWith("/")) {
-                    InputStream is = jar.getInputStream(entry);
-                    FileOutputStream fos = new FileOutputStream(f);
-
-                    // write contents of 'is' to 'fos'
-                    while (is.available() > 0) {
-                        fos.write(is.read());
-                    }
-
-                    fos.close();
-                    is.close();
-                }
-            }
-        }
-
-        String manifestPath = destinationDir + "/META-INF/MANIFEST.MF";
-        Manifest manifest = new Manifest(new URL("file:///" + manifestPath).openStream());
+        Manifest manifest = jar.getManifest();
         resp = manifest.getMainAttributes().getValue("Implementation-Version");
+        System.out.println("Found version: " + resp);
         return resp;
     }
 }
-- 
GitLab