Change online cog to use upserts set to ignore conflicts.

bot/cogs/online.py

@@ -32,17 +32,18 @@ async def on_member_update(self, before, after):
 
             # Try to insert the fact that the user is online. Date is autogenerated when this is run,
             # so is the primary key. There is a unique constraint on userid + date, so it won't let
-            # this happen twice. Doesn't matter though, so we just let the error happen.
-            session = (
-                session_creator()
+            # this happen twice.
+            session = session_creator()
+            # Might need to optimize sessions more later, this isn't that efficient
+            # Looks like I needed to after all, sessions were not being closed when the add errored out.
+            insert_stmt = insert(Online_Table).values(
+                discord_user_id=after.id,
+                online=True
+            ).on_conflict_do_nothing(
+                constraint="unique_user_per_day_constraint"
             )
-            try:
-                # Might need to optimize sessions more later, this isn't that efficient
-                # Looks like I needed to after all, sessions were not being closed when the add errored out.
-                session.add(Online_Table(discord_user_id=after.id, online=True))
-                session.commit()
-            except IntegrityError:
-                logging.info(f"{after.name} changed state but was already logged, skipping.")
+            session.execute(insert_stmt)
+            session.commit()
             session.close()
 
 

database/models.py

@@ -100,3 +100,6 @@ def as_dict(self):
 def session_creator() -> Session:
     session = sessionmaker(bind=engine)
     return session()
+
+
+global_session: Session = session_creator()