datastore: reduce DAO abstraction to remove complexity
StatementList and StatementElement were an attempt at creating our own ORM for JAMS. However, doing an database query was very verbose on the caller side, and if the 1st, 2nd or 4th field of StatementElement was user input, we would be vulnerable to SQL injections. This patch hardcoded the SQL queries in the DAO, making it easier on SQL expert to audit the queries. Change-Id: I4699bf3146a077efe03efd178b6190dfe60c72e9
Showing
- ad-connector/src/main/java/net/jami/jams/ad/connector/ADConnector.java 2 additions, 3 deletions...src/main/java/net/jami/jams/ad/connector/ADConnector.java
- ad-connector/src/main/java/net/jami/jams/ad/connector/service/UserProfileService.java 1 addition, 8 deletions...et/jami/jams/ad/connector/service/UserProfileService.java
- authentication-module/src/main/java/net/jami/jams/authmodule/TokenController.java 0 additions, 1 deletion...c/main/java/net/jami/jams/authmodule/TokenController.java
- authentication-module/src/main/java/net/jami/jams/authmodule/UserAuthenticationModule.java 11 additions, 25 deletions...va/net/jami/jams/authmodule/UserAuthenticationModule.java
- datastore/src/main/java/net/jami/datastore/dao/AbstractDao.java 61 additions, 37 deletions...ore/src/main/java/net/jami/datastore/dao/AbstractDao.java
- datastore/src/main/java/net/jami/datastore/dao/ContactDao.java 13 additions, 35 deletions...tore/src/main/java/net/jami/datastore/dao/ContactDao.java
- datastore/src/main/java/net/jami/datastore/dao/DeviceDao.java 24 additions, 46 deletions...store/src/main/java/net/jami/datastore/dao/DeviceDao.java
- datastore/src/main/java/net/jami/datastore/dao/GroupDao.java 20 additions, 62 deletionsdatastore/src/main/java/net/jami/datastore/dao/GroupDao.java
- datastore/src/main/java/net/jami/datastore/dao/JwtDao.java 0 additions, 71 deletionsdatastore/src/main/java/net/jami/datastore/dao/JwtDao.java
- datastore/src/main/java/net/jami/datastore/dao/PolicyDao.java 31 additions, 61 deletions...store/src/main/java/net/jami/datastore/dao/PolicyDao.java
- datastore/src/main/java/net/jami/datastore/dao/SystemDao.java 14 additions, 26 deletions...store/src/main/java/net/jami/datastore/dao/SystemDao.java
- datastore/src/main/java/net/jami/datastore/dao/UserDao.java 34 additions, 64 deletionsdatastore/src/main/java/net/jami/datastore/dao/UserDao.java
- datastore/src/main/java/net/jami/datastore/dao/UserGroupMappingsDao.java 30 additions, 54 deletions...ain/java/net/jami/datastore/dao/UserGroupMappingsDao.java
- datastore/src/main/java/net/jami/datastore/dao/UserProfileDao.java 72 additions, 44 deletions.../src/main/java/net/jami/datastore/dao/UserProfileDao.java
- datastore/src/main/java/net/jami/datastore/main/DataStore.java 14 additions, 55 deletions...tore/src/main/java/net/jami/datastore/main/DataStore.java
- datastore/src/test/java/net/jami/datastore/dao/DAOTest.java 5 additions, 15 deletionsdatastore/src/test/java/net/jami/datastore/dao/DAOTest.java
- jami-nameserver/src/main/java/net/jami/jams/nameserver/LocalNameServer.java 10 additions, 18 deletions...c/main/java/net/jami/jams/nameserver/LocalNameServer.java
- jams-ca/src/main/java/net/jami/jams/ca/workers/csr/builders/UserBuilder.java 10 additions, 11 deletions...va/net/jami/jams/ca/workers/csr/builders/UserBuilder.java
- jams-ca/src/main/java/net/jami/jams/ca/workers/ocsp/OCSPWorker.java 1 addition, 1 deletion...c/main/java/net/jami/jams/ca/workers/ocsp/OCSPWorker.java
- jams-common/src/main/java/net/jami/jams/common/dao/DeleteStatementBuilder.java 0 additions, 61 deletions...java/net/jami/jams/common/dao/DeleteStatementBuilder.java
Loading
Please register or sign in to comment