Core: Avoids a potential race condition for messages tied to a session

TYPE: Bugfix
LINK: ONEGOV-CLOUD-5DM

Author: Daverball

do/changes

@@ -158,8 +158,7 @@ class ChangelogCommit:
 
         if link.startswith('ONEGOV-CLOUD-'):
             return (
-                f'https://sentry.io/organizations/seantis-gmbh'
-                f'/issues/?query={link}'
+                f'https://seantis-gmbh.sentry.io/issues/?query={link}'
             )
 
         if link.startswith('STAKA-'):

src/onegov/core/request.py

@@ -621,15 +621,10 @@ def message(self, text: str, type: MessageType) -> None:
         template using the messages should call :meth:`consume_messages`.
 
         """
-        if not self.browser_session.has('messages'):
-            self.browser_session.messages = [Message(text, type)]
-        else:
-            # this is a bit akward, but I don't see an easy way for this atm.
-            # (otoh, usually there's going to be one message only)
-            self.browser_session.messages = [
-                *self.browser_session.messages,
-                Message(text, type)
-            ]
+        self.browser_session.messages = [
+            *self.browser_session.get('messages', ()),
+            Message(text, type)
+        ]
 
     def consume_messages(self) -> Iterator[Message]:
         """ Returns the messages, removing them from the session in the