Allowing to log SObject without ID field

Fixes #911

The `try` approach seems to run about two orders of magnitude faster than querying the Schema (`record.getSObjectType().getDescribe().fields.getMap().containsKey('Id')`) and one order of magnitude faster than cached version of the Schema (`private static final Map<String, Boolean> CACHED_SOBJECT_NAME_TO_HAS_ID_FIELD ...`)

Author: TrangOul

nebula-logger/core/main/logger-engine/classes/LogEntryEventBuilder.cls

@@ -460,7 +460,11 @@ global with sharing class LogEntryEventBuilder {
       this.logEntryEvent.RecordSObjectClassification__c = 'Unknown';
       this.logEntryEvent.RecordSObjectType__c = 'Unknown';
     } else {
-      this.logEntryEvent.RecordId__c = record.Id;
+      try {
+        this.logEntryEvent.RecordId__c = record.Id;
+      } catch (System.SObjectException ex) {
+        // Certain SObjects (such as Platform Events) do not have the ID field
+      }
       this.logEntryEvent.RecordSObjectClassification__c = getSObjectClassification(record.getSObjectType());
       this.logEntryEvent.RecordSObjectType__c = record.getSObjectType().toString();
       this.logEntryEvent.RecordSObjectTypeNamespace__c = getSObjectTypeNamespace(record.getSObjectType());