Merge pull request #422 from hargata/Hargata/auto.oidc

Enable auto login via OIDC

.gitignore

@@ -11,3 +11,4 @@ wwwroot/translations/
 config/userConfig.json
 CarCareTracker.csproj.user
 Properties/launchSettings.json
+data/cartracker-log.db

Controllers/APIController.cs

@@ -1,5 +1,4 @@
-using CarCareTracker.External.Implementations;
-using CarCareTracker.External.Interfaces;
+using CarCareTracker.External.Interfaces;
 using CarCareTracker.Filter;
 using CarCareTracker.Helper;
 using CarCareTracker.Logic;

Controllers/LoginController.cs

@@ -29,6 +29,18 @@ namespace CarCareTracker.Controllers
         }
         public IActionResult Index(string redirectURL = "")
         {
+            var remoteAuthConfig = _config.GetOpenIDConfig();
+            if (remoteAuthConfig.DisableRegularLogin && !string.IsNullOrWhiteSpace(remoteAuthConfig.LogOutURL))
+            {
+                var generatedState = Guid.NewGuid().ToString().Substring(0, 8);
+                remoteAuthConfig.State = generatedState;
+                if (remoteAuthConfig.ValidateState)
+                {
+                    Response.Cookies.Append("OIDC_STATE", remoteAuthConfig.State, new CookieOptions { Expires = new DateTimeOffset(DateTime.Now.AddMinutes(5)) });
+                }
+                var remoteAuthURL = remoteAuthConfig.RemoteAuthURL;
+                return Redirect(remoteAuthURL);
+            }
             return View(model: redirectURL);
         }
         public IActionResult Registration()
@@ -248,7 +260,12 @@ namespace CarCareTracker.Controllers
         public IActionResult LogOut()
         {
             Response.Cookies.Delete("ACCESS_TOKEN");
-            return Json(true);
+            var remoteAuthConfig = _config.GetOpenIDConfig();
+            if (remoteAuthConfig.DisableRegularLogin && !string.IsNullOrWhiteSpace(remoteAuthConfig.LogOutURL))
+            {
+                return Json(remoteAuthConfig.LogOutURL);
+            }
+            return Json("/Login");
         }
     }
 }

Controllers/VehicleController.cs

@@ -1,6 +1,5 @@
 using CarCareTracker.External.Interfaces;
 using CarCareTracker.Models;
-using LiteDB;
 using Microsoft.AspNetCore.Mvc;
 using CarCareTracker.Helper;
 using CsvHelper;

External/Implementations/Litedb/CollisionRecordDataAccess.cs

@@ -7,51 +7,48 @@ namespace CarCareTracker.External.Implementations
 {
     public class CollisionRecordDataAccess : ICollisionRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "collisionrecords";
+        public CollisionRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<CollisionRecord> GetCollisionRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<CollisionRecord>(tableName);
-                var collisionRecords = table.Find(Query.EQ(nameof(CollisionRecord.VehicleId), vehicleId));
-                return collisionRecords.ToList() ?? new List<CollisionRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<CollisionRecord>(tableName);
+            var collisionRecords = table.Find(Query.EQ(nameof(CollisionRecord.VehicleId), vehicleId));
+            return collisionRecords.ToList() ?? new List<CollisionRecord>();
         }
         public CollisionRecord GetCollisionRecordById(int collisionRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<CollisionRecord>(tableName);
-                return table.FindById(collisionRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<CollisionRecord>(tableName);
+            return table.FindById(collisionRecordId);
         }
         public bool DeleteCollisionRecordById(int collisionRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<CollisionRecord>(tableName);
-                table.Delete(collisionRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<CollisionRecord>(tableName);
+            table.Delete(collisionRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveCollisionRecordToVehicle(CollisionRecord collisionRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<CollisionRecord>(tableName);
-                table.Upsert(collisionRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<CollisionRecord>(tableName);
+            table.Upsert(collisionRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllCollisionRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<CollisionRecord>(tableName);
-                var collisionRecords = table.DeleteMany(Query.EQ(nameof(CollisionRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<CollisionRecord>(tableName);
+            var collisionRecords = table.DeleteMany(Query.EQ(nameof(CollisionRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/ExtraFieldDataAccess.cs

@@ -1,30 +1,30 @@
 using CarCareTracker.External.Interfaces;
 using CarCareTracker.Helper;
 using CarCareTracker.Models;
-using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
-    public class ExtraFieldDataAccess: IExtraFieldDataAccess
+    public class ExtraFieldDataAccess : IExtraFieldDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "extrafields";
+        public ExtraFieldDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public RecordExtraField GetExtraFieldsById(int importMode)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<RecordExtraField>(tableName);
-                return table.FindById(importMode) ?? new RecordExtraField();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<RecordExtraField>(tableName);
+            return table.FindById(importMode) ?? new RecordExtraField();
         }
         public bool SaveExtraFields(RecordExtraField record)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<RecordExtraField>(tableName);
-                table.Upsert(record);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<RecordExtraField>(tableName);
+            table.Upsert(record);
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/GasRecordDataAccess.cs

@@ -5,53 +5,50 @@ using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
-    public class GasRecordDataAccess: IGasRecordDataAccess
+    public class GasRecordDataAccess : IGasRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "gasrecords";
+        public GasRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<GasRecord> GetGasRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<GasRecord>(tableName);
-                var gasRecords = table.Find(Query.EQ(nameof(GasRecord.VehicleId), vehicleId));
-                return gasRecords.ToList() ?? new List<GasRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<GasRecord>(tableName);
+            var gasRecords = table.Find(Query.EQ(nameof(GasRecord.VehicleId), vehicleId));
+            return gasRecords.ToList() ?? new List<GasRecord>();
         }
         public GasRecord GetGasRecordById(int gasRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<GasRecord>(tableName);
-                return table.FindById(gasRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<GasRecord>(tableName);
+            return table.FindById(gasRecordId);
         }
         public bool DeleteGasRecordById(int gasRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<GasRecord>(tableName);
-                table.Delete(gasRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<GasRecord>(tableName);
+            table.Delete(gasRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveGasRecordToVehicle(GasRecord gasRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<GasRecord>(tableName);
-                table.Upsert(gasRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<GasRecord>(tableName);
+            table.Upsert(gasRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllGasRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<GasRecord>(tableName);
-                var gasRecords = table.DeleteMany(Query.EQ(nameof(GasRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<GasRecord>(tableName);
+            var gasRecords = table.DeleteMany(Query.EQ(nameof(GasRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/NoteDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
-    public class NoteDataAccess: INoteDataAccess
+    public class NoteDataAccess : INoteDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "notes";
+        public NoteDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<Note> GetNotesByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Note>(tableName);
-                var noteToReturn = table.Find(Query.EQ(nameof(Note.VehicleId), vehicleId));
-                return noteToReturn.ToList() ?? new List<Note>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Note>(tableName);
+            var noteToReturn = table.Find(Query.EQ(nameof(Note.VehicleId), vehicleId));
+            return noteToReturn.ToList() ?? new List<Note>();
         }
         public Note GetNoteById(int noteId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Note>(tableName);
-                return table.FindById(noteId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Note>(tableName);
+            return table.FindById(noteId);
         }
         public bool SaveNoteToVehicle(Note note)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Note>(tableName);
-                table.Upsert(note);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Note>(tableName);
+            table.Upsert(note);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteNoteById(int noteId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Note>(tableName);
-                table.Delete(noteId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Note>(tableName);
+            table.Delete(noteId);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllNotesByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Note>(tableName);
-                var notes = table.DeleteMany(Query.EQ(nameof(Note.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Note>(tableName);
+            var notes = table.DeleteMany(Query.EQ(nameof(Note.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/OdometerRecordDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
     public class OdometerRecordDataAccess : IOdometerRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "odometerrecords";
+        public OdometerRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<OdometerRecord> GetOdometerRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<OdometerRecord>(tableName);
-                var odometerRecords = table.Find(Query.EQ(nameof(OdometerRecord.VehicleId), vehicleId));
-                return odometerRecords.ToList() ?? new List<OdometerRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<OdometerRecord>(tableName);
+            var odometerRecords = table.Find(Query.EQ(nameof(OdometerRecord.VehicleId), vehicleId));
+            return odometerRecords.ToList() ?? new List<OdometerRecord>();
         }
         public OdometerRecord GetOdometerRecordById(int odometerRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<OdometerRecord>(tableName);
-                return table.FindById(odometerRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<OdometerRecord>(tableName);
+            return table.FindById(odometerRecordId);
         }
         public bool DeleteOdometerRecordById(int odometerRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<OdometerRecord>(tableName);
-                table.Delete(odometerRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<OdometerRecord>(tableName);
+            table.Delete(odometerRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveOdometerRecordToVehicle(OdometerRecord odometerRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<OdometerRecord>(tableName);
-                table.Upsert(odometerRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<OdometerRecord>(tableName);
+            table.Upsert(odometerRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllOdometerRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<OdometerRecord>(tableName);
-                var odometerRecords = table.DeleteMany(Query.EQ(nameof(OdometerRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<OdometerRecord>(tableName);
+            var odometerRecords = table.DeleteMany(Query.EQ(nameof(OdometerRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/PlanRecordDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
     public class PlanRecordDataAccess : IPlanRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "planrecords";
+        public PlanRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<PlanRecord> GetPlanRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecord>(tableName);
-                var planRecords = table.Find(Query.EQ(nameof(PlanRecord.VehicleId), vehicleId));
-                return planRecords.ToList() ?? new List<PlanRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecord>(tableName);
+            var planRecords = table.Find(Query.EQ(nameof(PlanRecord.VehicleId), vehicleId));
+            return planRecords.ToList() ?? new List<PlanRecord>();
         }
         public PlanRecord GetPlanRecordById(int planRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecord>(tableName);
-                return table.FindById(planRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecord>(tableName);
+            return table.FindById(planRecordId);
         }
         public bool DeletePlanRecordById(int planRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecord>(tableName);
-                table.Delete(planRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecord>(tableName);
+            table.Delete(planRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SavePlanRecordToVehicle(PlanRecord planRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecord>(tableName);
-                table.Upsert(planRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecord>(tableName);
+            table.Upsert(planRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllPlanRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecord>(tableName);
-                var planRecords = table.DeleteMany(Query.EQ(nameof(PlanRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecord>(tableName);
+            var planRecords = table.DeleteMany(Query.EQ(nameof(PlanRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/PlanRecordTemplateDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
     public class PlanRecordTemplateDataAccess : IPlanRecordTemplateDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "planrecordtemplates";
+        public PlanRecordTemplateDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<PlanRecordInput> GetPlanRecordTemplatesByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecordInput>(tableName);
-                var planRecords = table.Find(Query.EQ(nameof(PlanRecordInput.VehicleId), vehicleId));
-                return planRecords.ToList() ?? new List<PlanRecordInput>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecordInput>(tableName);
+            var planRecords = table.Find(Query.EQ(nameof(PlanRecordInput.VehicleId), vehicleId));
+            return planRecords.ToList() ?? new List<PlanRecordInput>();
         }
         public PlanRecordInput GetPlanRecordTemplateById(int planRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecordInput>(tableName);
-                return table.FindById(planRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecordInput>(tableName);
+            return table.FindById(planRecordId);
         }
         public bool DeletePlanRecordTemplateById(int planRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecordInput>(tableName);
-                table.Delete(planRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecordInput>(tableName);
+            table.Delete(planRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SavePlanRecordTemplateToVehicle(PlanRecordInput planRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecordInput>(tableName);
-                table.Upsert(planRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecordInput>(tableName);
+            table.Upsert(planRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllPlanRecordTemplatesByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<PlanRecord>(tableName);
-                var planRecords = table.DeleteMany(Query.EQ(nameof(PlanRecordInput.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<PlanRecord>(tableName);
+            var planRecords = table.DeleteMany(Query.EQ(nameof(PlanRecordInput.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/ReminderRecordDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
     public class ReminderRecordDataAccess : IReminderRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "reminderrecords";
+        public ReminderRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<ReminderRecord> GetReminderRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ReminderRecord>(tableName);
-                var reminderRecords = table.Find(Query.EQ(nameof(ReminderRecord.VehicleId), vehicleId));
-                return reminderRecords.ToList() ?? new List<ReminderRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ReminderRecord>(tableName);
+            var reminderRecords = table.Find(Query.EQ(nameof(ReminderRecord.VehicleId), vehicleId));
+            return reminderRecords.ToList() ?? new List<ReminderRecord>();
         }
         public ReminderRecord GetReminderRecordById(int reminderRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ReminderRecord>(tableName);
-                return table.FindById(reminderRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ReminderRecord>(tableName);
+            return table.FindById(reminderRecordId);
         }
         public bool DeleteReminderRecordById(int reminderRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ReminderRecord>(tableName);
-                table.Delete(reminderRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ReminderRecord>(tableName);
+            table.Delete(reminderRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveReminderRecordToVehicle(ReminderRecord reminderRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ReminderRecord>(tableName);
-                table.Upsert(reminderRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ReminderRecord>(tableName);
+            table.Upsert(reminderRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllReminderRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ReminderRecord>(tableName);
-                var reminderRecords = table.DeleteMany(Query.EQ(nameof(ReminderRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ReminderRecord>(tableName);
+            var reminderRecords = table.DeleteMany(Query.EQ(nameof(ReminderRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/ServiceRecordDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
-    public class ServiceRecordDataAccess: IServiceRecordDataAccess
+    public class ServiceRecordDataAccess : IServiceRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "servicerecords";
+        public ServiceRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<ServiceRecord> GetServiceRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ServiceRecord>(tableName);
-                var serviceRecords = table.Find(Query.EQ(nameof(ServiceRecord.VehicleId), vehicleId));
-                return serviceRecords.ToList() ?? new List<ServiceRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ServiceRecord>(tableName);
+            var serviceRecords = table.Find(Query.EQ(nameof(ServiceRecord.VehicleId), vehicleId));
+            return serviceRecords.ToList() ?? new List<ServiceRecord>();
         }
         public ServiceRecord GetServiceRecordById(int serviceRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ServiceRecord>(tableName);
-                return table.FindById(serviceRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ServiceRecord>(tableName);
+            return table.FindById(serviceRecordId);
         }
         public bool DeleteServiceRecordById(int serviceRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ServiceRecord>(tableName);
-                table.Delete(serviceRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ServiceRecord>(tableName);
+            table.Delete(serviceRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveServiceRecordToVehicle(ServiceRecord serviceRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ServiceRecord>(tableName);
-                table.Upsert(serviceRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ServiceRecord>(tableName);
+            table.Upsert(serviceRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllServiceRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<ServiceRecord>(tableName);
-                var serviceRecords = table.DeleteMany(Query.EQ(nameof(ServiceRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<ServiceRecord>(tableName);
+            var serviceRecords = table.DeleteMany(Query.EQ(nameof(ServiceRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/SupplyRecordDataAccess.cs

@@ -7,51 +7,48 @@ namespace CarCareTracker.External.Implementations
 {
     public class SupplyRecordDataAccess : ISupplyRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "supplyrecords";
+        public SupplyRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<SupplyRecord> GetSupplyRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<SupplyRecord>(tableName);
-                var supplyRecords = table.Find(Query.EQ(nameof(SupplyRecord.VehicleId), vehicleId));
-                return supplyRecords.ToList() ?? new List<SupplyRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<SupplyRecord>(tableName);
+            var supplyRecords = table.Find(Query.EQ(nameof(SupplyRecord.VehicleId), vehicleId));
+            return supplyRecords.ToList() ?? new List<SupplyRecord>();
         }
         public SupplyRecord GetSupplyRecordById(int supplyRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<SupplyRecord>(tableName);
-                return table.FindById(supplyRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<SupplyRecord>(tableName);
+            return table.FindById(supplyRecordId);
         }
         public bool DeleteSupplyRecordById(int supplyRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<SupplyRecord>(tableName);
-                table.Delete(supplyRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<SupplyRecord>(tableName);
+            table.Delete(supplyRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveSupplyRecordToVehicle(SupplyRecord supplyRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<SupplyRecord>(tableName);
-                table.Upsert(supplyRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<SupplyRecord>(tableName);
+            table.Upsert(supplyRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllSupplyRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<SupplyRecord>(tableName);
-                var supplyRecords = table.DeleteMany(Query.EQ(nameof(SupplyRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<SupplyRecord>(tableName);
+            var supplyRecords = table.DeleteMany(Query.EQ(nameof(SupplyRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/TaxRecordDataAccess.cs

@@ -1,57 +1,54 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
+using CarCareTracker.Helper;
 using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
     public class TaxRecordDataAccess : ITaxRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "taxrecords";
+        public TaxRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<TaxRecord> GetTaxRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<TaxRecord>(tableName);
-                var taxRecords = table.Find(Query.EQ(nameof(TaxRecord.VehicleId), vehicleId));
-                return taxRecords.ToList() ?? new List<TaxRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<TaxRecord>(tableName);
+            var taxRecords = table.Find(Query.EQ(nameof(TaxRecord.VehicleId), vehicleId));
+            return taxRecords.ToList() ?? new List<TaxRecord>();
         }
         public TaxRecord GetTaxRecordById(int taxRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<TaxRecord>(tableName);
-                return table.FindById(taxRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<TaxRecord>(tableName);
+            return table.FindById(taxRecordId);
         }
         public bool DeleteTaxRecordById(int taxRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<TaxRecord>(tableName);
-                table.Delete(taxRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<TaxRecord>(tableName);
+            table.Delete(taxRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveTaxRecordToVehicle(TaxRecord taxRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<TaxRecord>(tableName);
-                table.Upsert(taxRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<TaxRecord>(tableName);
+            table.Upsert(taxRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllTaxRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<TaxRecord>(tableName);
-                var taxRecords = table.DeleteMany(Query.EQ(nameof(TaxRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<TaxRecord>(tableName);
+            var taxRecords = table.DeleteMany(Query.EQ(nameof(TaxRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/TokenRecordDataAccess.cs

@@ -7,51 +7,47 @@ namespace CarCareTracker.External.Implementations
 {
     public class TokenRecordDataAccess : ITokenRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "tokenrecords";
+        public TokenRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<Token> GetTokens()
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Token>(tableName);
-                return table.FindAll().ToList();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Token>(tableName);
+            return table.FindAll().ToList();
         }
         public Token GetTokenRecordByBody(string tokenBody)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Token>(tableName);
-                var tokenRecord = table.FindOne(Query.EQ(nameof(Token.Body), tokenBody));
-                return tokenRecord ?? new Token();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Token>(tableName);
+            var tokenRecord = table.FindOne(Query.EQ(nameof(Token.Body), tokenBody));
+            return tokenRecord ?? new Token();
         }
         public Token GetTokenRecordByEmailAddress(string emailAddress)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Token>(tableName);
-                var tokenRecord = table.FindOne(Query.EQ(nameof(Token.EmailAddress), emailAddress));
-                return tokenRecord ?? new Token();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Token>(tableName);
+            var tokenRecord = table.FindOne(Query.EQ(nameof(Token.EmailAddress), emailAddress));
+            return tokenRecord ?? new Token();
         }
         public bool CreateNewToken(Token token)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Token>(tableName);
-                table.Insert(token);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Token>(tableName);
+            table.Insert(token);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteToken(int tokenId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Token>(tableName);
-                table.Delete(tokenId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Token>(tableName);
+            table.Delete(tokenId);
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/UpgradeRecordDataAccess.cs

@@ -7,51 +7,48 @@ namespace CarCareTracker.External.Implementations
 {
     public class UpgradeRecordDataAccess : IUpgradeRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
+        public UpgradeRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         private static string tableName = "upgraderecords";
         public List<UpgradeRecord> GetUpgradeRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UpgradeRecord>(tableName);
-                var upgradeRecords = table.Find(Query.EQ(nameof(UpgradeRecord.VehicleId), vehicleId));
-                return upgradeRecords.ToList() ?? new List<UpgradeRecord>();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UpgradeRecord>(tableName);
+            var upgradeRecords = table.Find(Query.EQ(nameof(UpgradeRecord.VehicleId), vehicleId));
+            return upgradeRecords.ToList() ?? new List<UpgradeRecord>();
         }
         public UpgradeRecord GetUpgradeRecordById(int upgradeRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UpgradeRecord>(tableName);
-                return table.FindById(upgradeRecordId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UpgradeRecord>(tableName);
+            return table.FindById(upgradeRecordId);
         }
         public bool DeleteUpgradeRecordById(int upgradeRecordId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UpgradeRecord>(tableName);
-                table.Delete(upgradeRecordId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UpgradeRecord>(tableName);
+            table.Delete(upgradeRecordId);
+            db.Checkpoint();
+            return true;
         }
         public bool SaveUpgradeRecordToVehicle(UpgradeRecord upgradeRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UpgradeRecord>(tableName);
-                table.Upsert(upgradeRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UpgradeRecord>(tableName);
+            table.Upsert(upgradeRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteAllUpgradeRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UpgradeRecord>(tableName);
-                var upgradeRecords = table.DeleteMany(Query.EQ(nameof(UpgradeRecord.VehicleId), vehicleId));
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UpgradeRecord>(tableName);
+            var upgradeRecords = table.DeleteMany(Query.EQ(nameof(UpgradeRecord.VehicleId), vehicleId));
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/UserAccessDataAcces.cs

@@ -7,8 +7,12 @@ namespace CarCareTracker.External.Implementations
 {
     public class UserAccessDataAccess : IUserAccessDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "useraccessrecords";
+        public UserAccessDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         /// <summary>
         /// Gets a list of vehicles user have access to.
         /// </summary>
@@ -16,45 +20,37 @@ namespace CarCareTracker.External.Implementations
         /// <returns></returns>
         public List<UserAccess> GetUserAccessByUserId(int userId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                return table.Find(x=>x.Id.UserId == userId).ToList();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            return table.Find(x => x.Id.UserId == userId).ToList();
         }
         public UserAccess GetUserAccessByVehicleAndUserId(int userId, int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                return table.Find(x => x.Id.UserId == userId && x.Id.VehicleId == vehicleId).FirstOrDefault();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            return table.Find(x => x.Id.UserId == userId && x.Id.VehicleId == vehicleId).FirstOrDefault();
         }
         public List<UserAccess> GetUserAccessByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                return table.Find(x => x.Id.VehicleId == vehicleId).ToList();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            return table.Find(x => x.Id.VehicleId == vehicleId).ToList();
         }
         public bool SaveUserAccess(UserAccess userAccess)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                table.Upsert(userAccess);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            table.Upsert(userAccess);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteUserAccess(int userId, int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                table.DeleteMany(x => x.Id.UserId == userId && x.Id.VehicleId == vehicleId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            table.DeleteMany(x => x.Id.UserId == userId && x.Id.VehicleId == vehicleId);
+            db.Checkpoint();
+            return true;
         }
         /// <summary>
         /// Delete all access records when a vehicle is deleted.
@@ -63,12 +59,11 @@ namespace CarCareTracker.External.Implementations
         /// <returns></returns>
         public bool DeleteAllAccessRecordsByVehicleId(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                table.DeleteMany(x=>x.Id.VehicleId == vehicleId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            table.DeleteMany(x => x.Id.VehicleId == vehicleId);
+            db.Checkpoint();
+            return true;
         }
         /// <summary>
         /// Delee all access records when a user is deleted.
@@ -77,12 +72,11 @@ namespace CarCareTracker.External.Implementations
         /// <returns></returns>
         public bool DeleteAllAccessRecordsByUserId(int userId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserAccess>(tableName);
-                table.DeleteMany(x => x.Id.UserId == userId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserAccess>(tableName);
+            table.DeleteMany(x => x.Id.UserId == userId);
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/UserConfigDataAccess.cs

@@ -1,39 +1,38 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
-using LiteDB;
+using CarCareTracker.Helper;
 
 namespace CarCareTracker.External.Implementations
 {
-    public class UserConfigDataAccess: IUserConfigDataAccess
+    public class UserConfigDataAccess : IUserConfigDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "userconfigrecords";
+        public UserConfigDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public UserConfigData GetUserConfig(int userId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserConfigData>(tableName);
-                return table.FindById(userId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserConfigData>(tableName);
+            return table.FindById(userId);
         }
         public bool SaveUserConfig(UserConfigData userConfigData)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserConfigData>(tableName);
-                table.Upsert(userConfigData);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserConfigData>(tableName);
+            table.Upsert(userConfigData);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteUserConfig(int userId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserConfigData>(tableName);
-                table.Delete(userId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserConfigData>(tableName);
+            table.Delete(userId);
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/UserRecordDataAccess.cs

@@ -1,66 +1,60 @@
 using CarCareTracker.External.Interfaces;
-using CarCareTracker.Helper;
 using CarCareTracker.Models;
 using LiteDB;
+using CarCareTracker.Helper;
 
 namespace CarCareTracker.External.Implementations
 {
     public class UserRecordDataAccess : IUserRecordDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "userrecords";
+        public UserRecordDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public List<UserData> GetUsers()
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserData>(tableName);
-                return table.FindAll().ToList();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserData>(tableName);
+            return table.FindAll().ToList();
         }
         public UserData GetUserRecordByUserName(string userName)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserData>(tableName);
-                var userRecord = table.FindOne(Query.EQ(nameof(UserData.UserName), userName));
-                return userRecord ?? new UserData();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserData>(tableName);
+            var userRecord = table.FindOne(Query.EQ(nameof(UserData.UserName), userName));
+            return userRecord ?? new UserData();
         }
         public UserData GetUserRecordByEmailAddress(string emailAddress)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserData>(tableName);
-                var userRecord = table.FindOne(Query.EQ(nameof(UserData.EmailAddress), emailAddress));
-                return userRecord ?? new UserData();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserData>(tableName);
+            var userRecord = table.FindOne(Query.EQ(nameof(UserData.EmailAddress), emailAddress));
+            return userRecord ?? new UserData();
         }
         public UserData GetUserRecordById(int userId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserData>(tableName);
-                var userRecord = table.FindById(userId);
-                return userRecord ?? new UserData();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserData>(tableName);
+            var userRecord = table.FindById(userId);
+            return userRecord ?? new UserData();
         }
         public bool SaveUserRecord(UserData userRecord)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserData>(tableName);
-                table.Upsert(userRecord);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserData>(tableName);
+            table.Upsert(userRecord);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteUserRecord(int userId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<UserData>(tableName);
-                table.Delete(userId);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<UserData>(tableName);
+            table.Delete(userId);
+            db.Checkpoint();
+            return true;
         }
     }
 }

External/Implementations/Litedb/VehicleDataAccess.cs

@@ -5,42 +5,41 @@ using LiteDB;
 
 namespace CarCareTracker.External.Implementations
 {
-    public class VehicleDataAccess: IVehicleDataAccess
+    public class VehicleDataAccess : IVehicleDataAccess
     {
-        private static string dbName = StaticHelper.DbName;
+        private ILiteDBHelper _liteDB { get; set; }
         private static string tableName = "vehicles";
+        public VehicleDataAccess(ILiteDBHelper liteDB)
+        {
+           _liteDB = liteDB;
+        }
         public bool SaveVehicle(Vehicle vehicle)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Vehicle>(tableName);
-                var result = table.Upsert(vehicle);
-                return true;
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Vehicle>(tableName);
+            var result = table.Upsert(vehicle);
+            db.Checkpoint();
+            return true;
         }
         public bool DeleteVehicle(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Vehicle>(tableName);
-                return table.Delete(vehicleId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Vehicle>(tableName);
+            var result = table.Delete(vehicleId);
+            db.Checkpoint();
+            return result;
         }
         public List<Vehicle> GetVehicles()
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Vehicle>(tableName);
-                return table.FindAll().ToList();
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Vehicle>(tableName);
+            return table.FindAll().ToList();
         }
         public Vehicle GetVehicleById(int vehicleId)
         {
-            using (var db = new LiteDatabase(dbName))
-            {
-                var table = db.GetCollection<Vehicle>(tableName);
-                return table.FindById(vehicleId);
-            };
+            var db = _liteDB.GetLiteDB();
+            var table = db.GetCollection<Vehicle>(tableName);
+            return table.FindById(vehicleId);
         }
     }
 }

Helper/FileHelper.cs

@@ -18,10 +18,12 @@ namespace CarCareTracker.Helper
     {
         private readonly IWebHostEnvironment _webEnv;
         private readonly ILogger<IFileHelper> _logger;
-        public FileHelper(IWebHostEnvironment webEnv, ILogger<IFileHelper> logger)
+        private ILiteDBHelper _liteDB;
+        public FileHelper(IWebHostEnvironment webEnv, ILogger<IFileHelper> logger, ILiteDBHelper liteDB)
         {
             _webEnv = webEnv;
             _logger = logger;
+            _liteDB = liteDB;
         }
         public List<string> GetLanguages()
         {
@@ -164,6 +166,8 @@ namespace CarCareTracker.Helper
                 }
                 if (File.Exists(dataPath))
                 {
+                    //Relinquish current DB file lock
+                    _liteDB.DisposeLiteDB();
                     //data path will always exist as it is created on startup if not.
                     File.Move(dataPath, StaticHelper.DbName, true);
                 }

Helper/LiteDBHelper.cs

@@ -0,0 +1,36 @@
+using LiteDB;
+
+namespace CarCareTracker.Helper;
+
+public interface ILiteDBHelper
+{
+    LiteDatabase GetLiteDB();
+    void DisposeLiteDB();
+}
+public class LiteDBHelper: ILiteDBHelper
+{
+    public LiteDatabase db { get; set; }
+    public LiteDBHelper()
+    {
+        if (db == null)
+        {
+            db = new LiteDatabase(StaticHelper.DbName);
+        }
+    }
+    public LiteDatabase GetLiteDB()
+    {
+        if (db == null)
+        {
+            db = new LiteDatabase(StaticHelper.DbName);
+        }
+        return db;
+    }
+    public void DisposeLiteDB()
+    {
+        if (db != null)
+        {
+            db.Dispose();
+            db = null;
+        }
+    }
+}

Helper/StaticHelper.cs

@@ -8,7 +8,7 @@ namespace CarCareTracker.Helper
     /// </summary>
     public static class StaticHelper
     {
-        public static string VersionNumber = "1.2.7";
+        public static string VersionNumber = "1.2.8";
         public static string DbName = "data/cartracker.db";
         public static string UserConfigPath = "config/userConfig.json";
         public static string GenericErrorMessage = "An error occurred, please try again later";

Models/OIDC/OpenIDConfig.cs

@@ -11,6 +11,8 @@
         public string Scope { get; set; }
         public string State { get; set; }
         public bool ValidateState { get; set; } = false;
+        public bool DisableRegularLogin { get; set; } = false;
+        public string LogOutURL { get; set; } = "";
         public string RemoteAuthURL { get { return $"{AuthURL}?client_id={ClientId}&response_type=code&redirect_uri={RedirectURL}&scope={Scope}&state={State}"; } }
     }
 }

Program.cs

@@ -16,6 +16,9 @@ StaticHelper.InitMessage(builder.Configuration);
 // Add services to the container.
 builder.Services.AddControllersWithViews();
 
+//LiteDB is always injected even if user uses Postgres.
+builder.Services.AddSingleton<ILiteDBHelper, LiteDBHelper>();
+
 //data access method
 if (!string.IsNullOrWhiteSpace(builder.Configuration["POSTGRES_CONNECTION"])){
     builder.Services.AddSingleton<IVehicleDataAccess, PGVehicleDataAccess>();
@@ -107,7 +110,21 @@ var app = builder.Build();
 
 // Configure the HTTP request pipeline.
 app.UseExceptionHandler("/Home/Error");
-app.UseStaticFiles();
+
+app.UseStaticFiles(new StaticFileOptions
+{
+    OnPrepareResponse = ctx =>
+    {
+        if (ctx.Context.Request.Path.StartsWithSegments("/images") || ctx.Context.Request.Path.StartsWithSegments("/documents"))
+        {
+            ctx.Context.Response.Headers.Add("Cache-Control", "no-store");
+            if (!ctx.Context.User.Identity.IsAuthenticated)
+            {
+                ctx.Context.Response.Redirect("/Login");
+            }
+        }
+    }
+});
 
 app.UseRouting();
 

Views/Vehicle/_CollisionRecords.cshtml

@@ -129,7 +129,19 @@
                         <td class="col-3 flex-grow-1 flex-shrink-1 text-truncate" data-column="notes">@CarCareTracker.Helper.StaticHelper.TruncateStrings(collisionRecord.Notes)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(collisionRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = collisionRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute))
+                                    {
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    }
+                                    else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

Views/Vehicle/_Gas.cshtml

@@ -199,7 +199,19 @@
                         <td class="col-3 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="notes">@StaticHelper.TruncateStrings(gasRecord.Notes)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(gasRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = gasRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute))
+                                    {
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    }
+                                    else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

Views/Vehicle/_OdometerRecords.cshtml

@@ -129,7 +129,19 @@
                         <td class="col-2 col-xl-3 flex-grow-1 flex-shrink-1 text-truncate" data-column="notes">@CarCareTracker.Helper.StaticHelper.TruncateStrings(odometerRecord.Notes, 75)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(odometerRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = odometerRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute))
+                                    {
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    }
+                                    else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

Views/Vehicle/_ServiceRecords.cshtml

@@ -129,7 +129,17 @@
                         <td class="col-3 text-truncate flex-grow-1 flex-shrink-1" data-column="notes">@CarCareTracker.Helper.StaticHelper.TruncateStrings(serviceRecord.Notes)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(serviceRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = serviceRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute)){
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    } else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

Views/Vehicle/_SupplyRecords.cshtml

@@ -145,7 +145,19 @@
                         <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" data-column="notes">@CarCareTracker.Helper.StaticHelper.TruncateStrings(supplyRecord.Notes)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(supplyRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = supplyRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute))
+                                    {
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    }
+                                    else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

Views/Vehicle/_TaxRecords.cshtml

@@ -121,7 +121,19 @@
                         <td class="col-3 flex-grow-1 flex-shrink-1 text-truncate" data-column="notes">@CarCareTracker.Helper.StaticHelper.TruncateStrings(taxRecord.Notes)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(taxRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = taxRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute))
+                                    {
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    }
+                                    else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

Views/Vehicle/_UpgradeRecords.cshtml

@@ -129,7 +129,19 @@
                         <td class="col-3 flex-grow-1 flex-shrink-1 text-truncate" data-column="notes">@CarCareTracker.Helper.StaticHelper.TruncateStrings(upgradeRecord.Notes)</td>
                         @foreach (string extraFieldColumn in extraFields)
                         {
-                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">@(upgradeRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "")</td>
+                            <td class="col-2 flex-grow-1 flex-shrink-1 text-truncate" style='display:none;' data-column="@extraFieldColumn">
+                                @{
+                                    var extraFieldValue = upgradeRecord.ExtraFields.Where(x => x.Name == extraFieldColumn)?.FirstOrDefault()?.Value ?? "";
+                                    if (!string.IsNullOrWhiteSpace(extraFieldValue) && Uri.IsWellFormedUriString(extraFieldValue, UriKind.Absolute))
+                                    {
+                                        <a href="@extraFieldValue" onclick="noPropagation()" target="_blank">@CarCareTracker.Helper.StaticHelper.TruncateStrings(extraFieldValue)</a>
+                                    }
+                                    else
+                                    {
+                                        @extraFieldValue
+                                    }
+                                }
+                            </td>
                         }
                     </tr>
                 }

wwwroot/js/garage.js

@@ -167,7 +167,7 @@ function initCalendar() {
 function performLogOut() {
     $.post('/Login/LogOut', function (data) {
         if (data) {
-            window.location.href = '/Login';
+            window.location.href = data;
         }
     })
 }

wwwroot/js/shared.js

@@ -1010,4 +1010,7 @@ function copyToClipboard(e) {
     var textToCopy = e.textContent.trim();
     navigator.clipboard.writeText(textToCopy);
     successToast("Copied to Clipboard");
+}
+function noPropagation() {
+    event.stopPropagation();
 }