fix: Record UTC timestamps for transactions (#547)

* fix: Record UTC timestamps for transactions

* fix: Do not upload coverage to zeus if no api token is available

* noop for bot

* syntax?

Author: untitaker

.travis.yml

@@ -62,7 +62,7 @@ script:
   - coverage combine .coverage*
   - coverage xml -i
   - codecov --file coverage.xml
-  - zeus upload -t "application/x-cobertura+xml" coverage.xml
+  - '[[ -z "$ZEUS_API_TOKEN" ]] || zeus upload -t "application/x-cobertura+xml" coverage.xml'
 
 notifications:
   webhooks:

sentry_sdk/tracing.py

@@ -133,7 +133,7 @@ def __init__(
         self.hub = hub
         self._tags = {}  # type: Dict[str, str]
         self._data = {}  # type: Dict[str, Any]
-        self.start_timestamp = datetime.now()
+        self.start_timestamp = datetime.utcnow()
 
         #: End timestamp of span
         self.timestamp = None  # type: Optional[datetime]
@@ -279,7 +279,7 @@ def finish(self, hub=None):
             # This transaction is already finished, so we should not flush it again.
             return None
 
-        self.timestamp = datetime.now()
+        self.timestamp = datetime.utcnow()
 
         _maybe_create_breadcrumbs_from_span(hub, self)
 

tests/test_transport.py

@@ -44,7 +44,7 @@ def test_transport_works(
     Hub.current.bind_client(client)
     request.addfinalizer(lambda: Hub.current.bind_client(None))
 
-    add_breadcrumb(level="info", message="i like bread", timestamp=datetime.now())
+    add_breadcrumb(level="info", message="i like bread", timestamp=datetime.utcnow())
     capture_message("löl")
 
     getattr(client, client_flush_method)()