Started 9 mo 0 days ago
Took 25 min on basic-tc0zc

Failed Build #84 (Oct 28, 2020, 12:40:39 PM)

Changes
  1. Issue #773: `ThingCommandEnforcement` now uses its own `DittoLogger` instead of the `DittoDiagnosticLoggingAdapter` of its parent. (details / githubweb)
  2. Issue #773: Got rid of unnecessary `Optional`s within `ThingCommandEnforcement` to reduce complexity of the class and thus improve understandability. (details / githubweb)
  3. Issue #773: Use DittoLogger to set correlation ID in `PreEnforcer`. (details / githubweb)
  4. Issue #773: Renamed variables and parameters to make code easier to understand. (details / githubweb)
  5. Issue #773: Introduced `ThreadSafeDittoLogger` type and an immutable implementation thereof. (details / githubweb)
  6. Issue #773: Added "mixin" interface `WithMdcEntry` to allow the setting of arbitrary MDC values to a logger. (details / githubweb)
  7. Issue #773: Use `ThreadSafeDittoLogger` in `ThingCommandEnforcement` and `MessageMappingProcessorActor`. (details / githubweb)
  8. Issue #773: Allow to set arbitrary context values to `DittoLogger` and some refactorings: (details / githubweb)
  9. Issue #773: Allow to set arbitrary context values to `DittoDiagnosticLoggingAdapter` and added unit tests. (details / githubweb)
  10. Issue #773: Removed superfluous method delegations from direct sub-classes of `AbstractDiagnostincLoggingAdapter`. (details / githubweb)
  11. Issue #773: Introduced `ThreadSafeDittoDiagnosticLoggingAdapter` as well as an immutable implementation thereof. (details / githubweb)
  12. Issue #773: Made `ThreadSafeDittoDiagnosticLoggingAdapter` a `LoggingAdapter` instead of a `DiagnosticLoggingAdapter` as setting the MDC directly would make immutable implementation impossible. Renamed the classes accordingly. Simplified `ImmutableDittoLoggingAdapter`. (details / githubweb)
  13. Issue #773: Renamed `ImmutableThreadSafeDittoLogger` to `ImmutableDittoLogger` to remove redundancy in class name. (details / githubweb)
  14. Issue #792: add scaffold for another DData for uniqueness constraint on ack labels; minor improvements. (details / githubweb)
  15. Issue #792: Add a distributed data without lossy compression for uniqueness constraint. (details / githubweb)
  16. Issue #792: Bootstrap AcksUpdater; remove terminated subscribers from DData. (details / githubweb)
  17. Issue #792: AcksUpdater: detect local losers of race conditions. (details / githubweb)
  18. Issue #792: remove SubUpdater dependency from AcksUpdater. (details / githubweb)
  19. Issue #792: allow ack label declarations at DistributedSub. (details / githubweb)
  20. Issue #773: Manually merged test cases from master branch. (details / githubweb)
  21. Issue #792: pubsub: extract ClusterMemberRemovedAware mixin; add seqNr to SubAck so that they are not sent out prematurely. (details / githubweb)
  22. Issue #792: move AcknowledgementLabelNotUniqueException to ditto.signal.acks.base. (details / githubweb)
  23. Issue #773: Create a copy of MDC when getting MDC from DiagnosticLoggingAdapter as it returns an immutable map. This differs from the actual documentation of Akka and caused exceptions. (details / githubweb)
  24. Issue #773: Use `ThreadSafeDittoLogger` for logging and some refactoring: (details / githubweb)
  25. Issue #773: Removed dedicated methods for setting and discarding correlation ID from `AbstractDiagnosticLoggingAdapter` as the more generic method 'putMdcEntry' can be used instead. (details / githubweb)
  26. Issue #773: Only make copy of MDC if is going to be changed. (details / githubweb)
  27. Issue #773: Use `ThreadSafeDittoLoggingAdapter` for logging in `BasePublisherActor` and `BaseClientActor` as well as in their sub-classes. (details / githubweb)
  28. Issue #773: Fixed unit test. (details / githubweb)
  29. Issue #773: Fixed unit tests. (details / githubweb)
  30. Issue #773: Fixed more unit tests. (details / githubweb)
  31. Issue #773: Fixed even more unit tests. (details / githubweb)
  32. Issue #773: review: scaladoc styleguide fixes (details / githubweb)
  33. Fix a concurrency issue in AcksUpdater. (details / githubweb)
  34. Issue #792: stop using localSubscriptionsChanged in AcksUpdater; add tests for race conditions; fix remote conflict test. (details / githubweb)
  35. Issue #773: fixed unit tests by fixing mocking of loggers (details / githubweb)
  36. Issue #792: simplify AcksUpdater; update documentation and test. (details / githubweb)
  37. Issue #792: make authentication non-blocking; update TODOs; make query parameters accessible to WebsocketRoute. (details / githubweb)
  38. Issue #773: Fixed compilation error: implementations have to use immutable Seq. (details / githubweb)
  39. Issue #792: Make ack label declaration optional in PubSubFactory; disconnect websocket on AcknowledgementLabelNotUniqueException. (details / githubweb)
  40. Issue #792: fix StreamingSessionActorHeaderInteractionTest; remove unused fields; fix compile errors. (details / githubweb)
  41. Issue #792: relax ack label regex. (details / githubweb)
  42. review of declared acks in pubsub (details / githubweb)
  43. added correlation-id when "retrievePartialThing" / enriching outbound signals in MessageMappingProcessorActor (details / githubweb)
  44. fixed adding correlation-id for "RetrieveThing" commands caused by enrichment (details / githubweb)
  45. Fixing dependecies (details / githubweb)
  46. Add implementation for ConnectionPlaceholder to resolve the ID (details / githubweb)
  47. Issue #773: Post-merger adjustments: (details / githubweb)
  48. Issue #792: Make -AutthorizationEnforcer asynchronous for Websocket and SSE. (details / githubweb)
  49. Issue #792: stop testing DummyQueryCommand. (details / githubweb)
  50. Issue #792: Add "declaredAcks" to connection sources. (details / githubweb)
  51. Issue #792: validate that issued and declared acks of connections are prefixed with the connection ID. (details / githubweb)
  52. Issue #792: maintain "ackLabelDeclared" flag in ConnectionPersistenceActor. (details / githubweb)
  53. Issue #792: test connection ack label declaration; fix tests. (details / githubweb)
  54. Issue #792: deny acknowledgements from WS and connections unless declared. (details / githubweb)
  55. Issue #792: ensure Ack.LabelNotDeclaredException gets published to reply targets; discriminate incoming acknowledgements by their source. (details / githubweb)
  56. Issue #792: fix MessageMappingProcessorActorHeaderInteractionTest; test StreamingSessionActor's acknowledgement handling. (details / githubweb)
  57. Issue #792: Test that WS streams complete when StreamingSessionActor stops. (details / githubweb)
  58. Use ConnectionIdPlaceholder in Resolvers (details / githubweb)
  59. Resolve connection ID placeholder in declared and issued acks (details / githubweb)
  60. Review of declaration of acks for websocket connections (details / githubweb)
  61. Issue #773: Post-merger adjustments: Fixed compilation errors. (details / githubweb)
  62. added metric about acknowledgements (whether they were a success and redeliver status) (details / githubweb)
  63. Do not resolve connection ID placeholder in source and target acknowledgements (details / githubweb)
  64. Issue #792: GlobalCommandRegistryTestCases: exclude Dummy* commands instead of private classes. (details / githubweb)
  65. Issue #792: Fix review findings and StreamingSessionActorTest. (details / githubweb)
  66. Handle case of empty acknowledgements list inside declareAcknowledgements (details / githubweb)
  67. Split MessageMappingProcessor into Inbound- and OutboundMappingProcessor (details / githubweb)
  68. Issue #773: Not-thread-safe loggers and logging adapters handle ConcurrentModificationException for cases when they are used by multiple threads. (details / githubweb)
  69. Issue #773: Directives are simultaneously executed by multiple threads, thus thread-safe logger has to be used. (details / githubweb)
  70. Separate inbound and outbound MessageMappingProcessorActors (details / githubweb)
  71. Issue #773: Logger of `WebSocketRoute` is simultaneously called by multiple threads, thus thread-safe logger is used. (details / githubweb)
  72. fix unit tests (details / githubweb)
  73. added 1.3 as new docVersion (details / githubweb)
  74. increased binary-compatibility-check version to released 1.3.0 (details / githubweb)
  75. Issue #773: Set version of `@since` tags to upcoming Ditto release 1.4.0. (details / githubweb)
  76. restructure documentation to bring default header values closer to their interaction description (details / githubweb)
  77. Issue #792: test target-issued acks handling by connection persistence actor and fix it. (details / githubweb)
  78. Do allow live message responses without enforcing technical acknowledgement (details / githubweb)
  79. Consider target ack to be requested if target ack is live-response (details / githubweb)
  80. Add implicit acknowledgement only if requested-acks is non-empty (details / githubweb)
  81. consider a live response with a non timeout status code to be successful (details / githubweb)
  82. fixed that search via SSE was broken (details / githubweb)
  83. decouple response-required default value from requested acks (details / githubweb)
  84. Adjust documentation for header interaction between requested-acks (details / githubweb)
  85. Adjust documentation regarding the default value of response-required (details / githubweb)
  86. review: fixed javadoc errors (details / githubweb)
  87. Fixes javadoc and removes telling the exception to a sender (details / githubweb)
  88. Issue #792: adjusted since javadoc annotations to 1.4.0 (details / githubweb)
  89. Issue #792: removed "declared-acks" from DittoHeaderDefinition and DittoHeaders again (details / githubweb)
  90. Issue #792: re-added "declared-acks" to DittoHeaderDefinition, but keep removed from DittoHeaders (details / githubweb)
  91. Issue #792: log and respond with errors when subscribing to Ditto pubsub (details / githubweb)
  92. Issue #792: post merge commit after merge from master (details / githubweb)
  93. Issue #792: used ConnectivityInternalErrorException instead of Gateway one for ConnectionPersistenceActor (details / githubweb)
  94. fixed long open TODO: made ConnectivityStatus Jsonifiable (details / githubweb)
  95. Issue #792: terminated session actor after subscribe to Ditto pubsub failed (details / githubweb)
  96. slight improvement on the wording in the policy base documentation (details / githubweb)
  97. Issue #773: Handle ConcurrentModificationException at all places where it could be thrown to avoid disturbing actual service processes. (details / githubweb)
  98. Issue #773: Use string for removal. (details / githubweb)
  99. Adds blog post about the acknowledgement feature (details / githubweb)
  100. Issue #773: Javadoc of java.util.Map does not fully reveal where ConcurrentModificationExceptions are thrown, thus catching the exception for all interactions with 'localMdc' map. (details / githubweb)
  101. relax acknowledgement label regex (details / githubweb)
  102. fix unit tests (details / githubweb)
  103. replace since 1.3.0 with 1.4.0 for newly introduced AcknowledgementLabelNotUniqueException (details / githubweb)
  104. completed javadoc todos (details / githubweb)
  105. Add blog post about async client creation; update email of authors. (details / githubweb)
  106. Update java client documentation. (details / githubweb)
  107. Issue #792: review of e2e-acknowledgement blogpost: (details / githubweb)
  108. Fixed Mapping timers (details / githubweb)
  109. log warning on expiration of mapping timer (details / githubweb)
  110. Validate connection ID (details / githubweb)
  111. adapt json example to described use case (details / githubweb)
  112. Issue #792: relaxed AcknowledgementLabel regex to be able to start with a placeholder (details / githubweb)
  113. Issue #792: concretize message mapping outcome.. (details / githubweb)
  114. Issue #792: remove self messaging of OutboudMappingProcessorActor; remove unnecessary parts of AbstractGraphActor; fix logger thread safety. (details / githubweb)
  115. Issue #792: separate inbound mapping processor actor into 2 actors. (details / githubweb)
  116. Issue #792: delete duplicate serialVersionUID from ConnectionIdInvalidException. (details / githubweb)
  117. Issue #792: Fix consumer actor tests and a NullPointerException in AmqpConsumerActor. (details / githubweb)
  118. Issue #792: document declared-acks. (details / githubweb)
  119. Issue #792: allow WebSocketAuthorizationEnforcer to extend DittoHeaders. (details / githubweb)
  120. Issue #792: fixed wrong logic in checking for fully resolved ack labels in DittoProtocolSubImpl (details / githubweb)
  121. Issue #792: Move algebraic datatype MappingOutcome to own package to enforce visibility. (details / githubweb)
  122. Issue #792: fixed resolving target issued ack labels where they were not yet resolved (details / githubweb)
  123. ConnectionIdPlaceholder now only accepts ConnectionIds (details / githubweb)
  124. Issue #792: review: add correlationId to logger where missing (details / githubweb)
  125. allow connection IDs with a lengt of 80 (details / githubweb)
  126. Issue #792: doc review: fixed some typos (details / githubweb)
  127. fix unit test (details / githubweb)
  128. For live signals it's required to restore command connectivity headers (details / githubweb)
  129. added IoT WG community day status update slides (details / githubweb)
  130. added some documentation about SSO with OpenID connect in the browser (details / githubweb)
  131. Issue #792: make global/fallback timeouts for AMQP 1.0 connections configurable (details / githubweb)
  132. Issue #792: exclude InboundDispatchingActor from receiving RetrieveAddressStatus messages (details / githubweb)
  133. Issue #792: monitor successfully and failed publishing of outbound signals (details / githubweb)
  134. Issue #792: fixed duplicated monitored failed publishing of outbound signals (details / githubweb)
  135. Issue #792: fixed duplicated monitored success publishing of outbound signals (dependent on whether an auto issued ack is created or not) (details / githubweb)
  136. Issue #792: fixed that a sent live message to a target containing an issued ack still contained that auto issued ack as "requested-acks" in its headers (details / githubweb)
  137. Issue #792: undid monitoring publish success when no commandResponse (ack or live) was created when publishing a signal (details / githubweb)
  138. Issue #792: fixed filtering of "requsted-acks" for connections without declared source acks (details / githubweb)
  139. Issue #792: reverted removing issued acks from header as this breaks auto ack issuing (details / githubweb)
  140. Issue #792: adjusted signal to be mapped to adaptable right before that step to filter out "issued acks" by targets from the "requested-acks" header (details / githubweb)
  141. Issue #792: Fix 500 for HTTP requests with invalid requested-acks. (details / githubweb)
  142. Issue #792: verify that query commands with ack requests do not cause redelivery. (details / githubweb)
  143. Test header resetting in Ack.Agg.Actor and fix it. (details / githubweb)
  144. Test recording of timer duration above 1s. (details / githubweb)
  145. Fix recording of timer duration above 1s. (details / githubweb)
  146. updated slide with todays GH stars (details / githubweb)
  147. Issue #792: removed HTTP headers from being added to messages sent via HTTP POST (details / githubweb)
  148. fixed using thread-safe loggers where we need to (details / githubweb)
  149. Issue #792: ensure AcknowledgementLabelNotDeclaredException is mapped with the correct topic path information. (details / githubweb)
  150. provide more explicit information in AcknowledgementLabelInvalidException (details / githubweb)
  151. Replace exclamation marks with dots + reduce streaming chaing (details / githubweb)
  152. Make base client actor more robust by catching DittoRuntimeExceptions (details / githubweb)
  153. Implementing RequestedAcksValueValidator to fail fast (details / githubweb)
  154. Issue #792: reuse AbstractHttpRequestActor.handleDittoRuntimeException; remove unused imports in HttpRequestActorTest. (details / githubweb)
  155. Issue #792: fix metric counting/logging inconsistencies (details / githubweb)
  156. Issue #792: removed unused method (details / githubweb)
  157. Wrap sending the Amqp messages in a stream with a preceding queue/buffer (details / githubweb)
  158. Make clear that WebSockets are not meant to be used for horizontal scaling (details / githubweb)
  159. Issue #792: fixed/adjusted acknowledgement logging texts for connectionLogger (details / githubweb)
  160. fixes #844: "location" header was not read when provided by external headers (details / githubweb)
  161. Issue #792: added MessageMapper id to connectionLog statements about mapped/dropped (and error) (details / githubweb)
  162. Issue #792: fixed mapping texts which had the wrong direction (details / githubweb)
  163. Make protocol of OAuth providers configurable. (details / githubweb)
  164. Set response-required to false for responses and events (details / githubweb)
  165. updated CQs (details / githubweb)
  166. Provide second constructor to allow providing a non static category for (details / githubweb)
  167. Issue #792: documentation review: added "warning" sections for clarifying that the WS is not intended to be used as scaling out or doing HA (details / githubweb)
  168. Issue #792: review: restore DittoHeaders on a best effort basis in InboundMappingProcessor in order to retain e.g. "reply-to" required to find out where to send an error response to (details / githubweb)
  169. Issue #792: Review: Add argument matchers to SendingTest. (details / githubweb)
  170. Add StoppedKamonTimer.getElapsedNano; fix logging. (details / githubweb)
  171. Issue #792: fixed merging of "best effort headers" with already existing DittoHeaders (details / githubweb)
  172. adjusted all DittoRuntimeExceptions to overwrite setDittoHeaders(DittoHeaders) method in order to preserve the type of the exception; unfortunately this can't be fixed without major changes to DittoRuntimeException (details / githubweb)
  173. #844: fixed header case sensitivity for CheckExternalFilter used in HeaderTranslator (details / githubweb)
  174. Issue #792: added best effort fallback for determining entity id for exceptions which did not yet pass the protocol adapter (details / githubweb)
  175. close streaming session when an invalid jwt is provided (details / githubweb)
  176. Add tests for new introduced publisher-queue (details / githubweb)
  177. fix javadoc (details / githubweb)
  178. code wrapping was required. added closing bracket (details / githubweb)
  179. Issue #792: adjusted blogpost date about acks to today (details / githubweb)
  180. Add JavaDocs and some minor refactorings of the newly integrated pusblisher-queue (details / githubweb)
  181. Introduce SilentlyFailingRevocationChecker (details / githubweb)
  182. Add the new config values to the new config tests, which were introduced in the last merge (details / githubweb)
  183. AmqpPublisherActorTest: Prevent test failure due to low timeout. (details / githubweb)
  184. Review: AmqpPublisherActor: move non-thread-safe producer lookup to the actor's thread; format code; fix comment; do not request ack for the congesting messages in AmqpPublisherActorTest. (details / githubweb)
  185. formatting (details / githubweb)
  186. fix test after merge (details / githubweb)
  187. Deprecate topic:entityId placeholder and introduce topic:entityName placeholder (details / githubweb)
  188. document new placeholder topic:entityName and deprecate topic:entityId (details / githubweb)
  189. Log handling of http response for http forwarding connection in connection logs (details / githubweb)
  190. Make revocation check optional at the trust-manager level. (details / githubweb)
  191. make connection logger a field of BasePublisherActor (details / githubweb)
  192. Make connectionLogger field in DittoTrustManager non nullable (details / githubweb)
  193. Make hostname non nullable since it was only null in test cases (details / githubweb)
  194. delegateWithoutRevocationCheck can never be null (details / githubweb)
  195. Add 1.4.0 release notes. (details / githubweb)

Started by user Yufei Cai

Revision: 7388c80dbb28e1a10fcffad59beb470227b5ec09
Repository: https://github.com/eclipse/ditto.git
  • origin/master
Test Result (1 failure / +1)