Refactor getConfigFieldsInfo method in Start and PostgreSQLConfig cla…

…sses
supertokens · Feb 23, 2024 · 6605a5d · 6605a5d
1 parent dedf48a
commit 6605a5d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/src/main/java/io/supertokens/storage/postgresql/Start.java b/src/main/java/io/supertokens/storage/postgresql/Start.java
@@ -2784,7 +2784,7 @@ public Set<String> getValidFieldsInConfig() {
     }
 
     @Override
-    public ArrayList<ConfigFieldInfo> getConfigFieldsInfo() throws InvalidConfigException {
+    public ArrayList<ConfigFieldInfo> getConfigFieldsInfo() {
         return PostgreSQLConfig.getConfigFieldsInfo();
     }
 

diff --git a/src/main/java/io/supertokens/storage/postgresql/config/PostgreSQLConfig.java b/src/main/java/io/supertokens/storage/postgresql/config/PostgreSQLConfig.java
@@ -159,7 +159,7 @@ public static Set<String> getValidFields() {
         return validFields;
     }
 
-    public static ArrayList<ConfigFieldInfo> getConfigFieldsInfo() throws InvalidConfigException {
+    public static ArrayList<ConfigFieldInfo> getConfigFieldsInfo() {
         ArrayList<ConfigFieldInfo> result = new ArrayList<ConfigFieldInfo>();
 
         for (String fieldId : PostgreSQLConfig.getValidFields()) {
@@ -185,10 +185,8 @@ public static ArrayList<ConfigFieldInfo> getConfigFieldsInfo() throws InvalidCon
                     type = "boolean";
                 } else if (fieldType == int.class || fieldType == long.class || fieldType == Integer.class) {
                     type = "number";
-                }
-
-                if (type == null) {
-                    throw new InvalidConfigException("Unsupported field type: " + fieldType.getName());
+                } else {
+                    throw new RuntimeException("Unknown field type " + fieldType.getName());
                 }
 
                 result.add(new ConfigFieldInfo(name, description, isDifferentAcrossTenants, type));