feat: addressing PR reviews

Author: mariajgrimaldi

openedx_events/tooling.py

@@ -4,6 +4,7 @@
 import socket
 import warnings
 from datetime import datetime
+from uuid import uuid1
 
 from django.conf import settings
 from django.dispatch import Signal
@@ -64,12 +65,17 @@ def generate_signal_metadata(self):
                 'sourcelib: '0.1.0',
             }
         """
+        def generate_uuid():
+            """
+            Generate events id based on UUID version 1.
+            """
+            return uuid1()
 
         def get_current_time():
             """
             Getter function used to get timestamp when the event ocurred.
             """
-            return datetime.now().strftime("%Y-%m-%dT%H:%M:%SZ")
+            return datetime.utcnow()
 
         def get_source():
             """
@@ -85,28 +91,19 @@ def get_source_host():
             """
             return socket.gethostname()
 
-        def get_spec_version():
-            """
-            Getter function used to obtain CloudEvents version.
-
-            This field is added to be compliant with OEP-41, it's not
-            necessarily significant to the Open edX events metadata.
-            """
-            return "1.0"
-
         def get_source_lib():
             """
             Getter function used to obtain Open edX Events version.
             """
             return openedx_events.__version__
 
         return {
+            "id": generate_uuid(),
             "event_type": self.event_type,
             "minorversion": self.minor_version,
             "time": get_current_time(),
             "source": get_source(),
             "sourcehost": get_source_host(),
-            "specversion": get_spec_version(),
             "sourcelib": get_source_lib(),
         }