Fix silent ignoring of superfluous save retries (#5780)

CHANGELOG.unreleased.md

@@ -24,6 +24,7 @@ For upgrade instructions, please check the [migration guide](MIGRATIONS.released
 - Jobs status is no longer polled if jobs are not enabled, avoiding backend logging spam [#5761](https://github.com/scalableminds/webknossos/pull/5761)
 - Fixed a bug that windows user could not open the context menu as it instantly closed after opening. [#5756](https://github.com/scalableminds/webknossos/pull/5756).
 - Fixed a bug where the health check of public datasets failed if no cookie/token was supplied. [#5768](https://github.com/scalableminds/webknossos/pull/5768).
+- Fixed a bug where retried save requests could lead to a 409 CONFLICT error if the first request was already handled by the back-end. [#5779](https://github.com/scalableminds/webknossos/pull/5779).
 
 ### Removed
 -

webknossos-tracingstore/app/com/scalableminds/webknossos/tracingstore/controllers/TracingController.scala

@@ -217,7 +217,7 @@ trait TracingController[T <: GeneratedMessage, Ts <: GeneratedMessage] extends C
     updateGroup.transactionId match {
       case Some(transactionId) =>
         for {
-          _ <- Fox.assertTrue(tracingService.handledGroupIdStoreContains(transactionId, tracingId, updateGroup.version)) ?~> errorMessage ~> CONFLICT
+          _ <- Fox.assertTrue(tracingService.handledGroupIdStoreContains(tracingId, transactionId, updateGroup.version)) ?~> errorMessage ~> CONFLICT
         } yield updateGroup.version
       case None => Fox.failure(errorMessage) ~> CONFLICT
     }