Merge pull request #303 from poissoncorp/RavenDB-26793

RavenDB-26793 Time-series bulk-insert UTC fix + custom-entity-name control-char guard

Author: poissoncorp

ravendb/tests/jvm_migrated_tests/client_tests/test_custom_entity_name.py

@@ -100,7 +100,9 @@ def _customize_store(self, store: DocumentStore) -> None:
         )
 
     def __test_when_collection_and_id_contain_special_chars(self, c: str) -> None:
-        if 14 <= ord(c) <= 31:
+        # RavenDB now rejects control characters in document ids (including the generated HiLo id),
+        # so skip the whole C0 control range rather than only 14..31.
+        if ord(c) <= 31:
             return
 
         self.c = c

ravendb/tools/utils.py

@@ -15,7 +15,7 @@
     from collections import Iterable, Sequence
 
 from ravendb.tools.projection import create_entity_with_mapper
-from datetime import datetime, timedelta
+from datetime import datetime, timedelta, timezone
 from enum import Enum
 from threading import Timer
 from copy import deepcopy
@@ -958,4 +958,8 @@ def add_minutes(date: datetime, minutes: int):
 
     @staticmethod
     def get_unix_time_in_ms(date: datetime) -> int:
+        # Naive datetimes are treated as UTC (consistent with Utils.datetime_to_string) so that
+        # bulk-insert time-series timestamps are not shifted by the machine's local UTC offset.
+        if date.tzinfo is None:
+            date = date.replace(tzinfo=timezone.utc)
         return int(date.timestamp() * 1000)