Error serializing UuidV7 during event sourcing after upgrading to Ecotone v1.256

[m2mtech]

Ecotone version(s) affected: 1.256.0

Description
When upgrading from v1.255 to v1.256, commands containing a UuidV7 parameter cause an error during serialization in event sourcing:

Error: Typed static property Symfony\Component\Uid\UuidV7::\$seedParts must not be accessed before initialization

https://github.com/m2mtech/ecotoneUuidProblem/blob/ea98ecd5c106f85a1ad32077e3311aa0f49ecd96/error.log#L1

This happens during event sourcing when a change command contains the projected and modified entity.

How to reproduce

1. Clone the test repository: ecotoneUuidProblem
2. Run:

   docker compose run php83 composer update
   docker compose run php83 bin/phpunit

Context

• PHP 8.3 (also tested with 8.4)
• Symfony 6.4 (also tested with 7.3)
• The issue was introduced with Ecotone v1.256
• No issue on v1.255

[lifinsky]

Hi. You have a bug in your event-sourced aggregate — you need to handle the ProductCreated event via an event sourcing handler and assign the product id (aggregate id) and name.

[lifinsky]

https://docs.ecotone.tech/modelling/event-sourcing/event-sourcing-introduction/event-sourcing-aggregates/different-ways-to-record-events

[m2mtech]

Thank you for the feedback. I have added te EventSourcingHandler for the ProductCreated event. The error persists. It is also implemented in the original code that showed the error.

[lifinsky]

@m2mtech You also need converter for UuidV7 class:
https://docs.ecotone.tech/modules/jms-converter

[m2mtech]

I did not need one for 1.255?

[lifinsky]

This is not an Ecotone issue, but rather a matter of bootstrapping the Symfony application in the test. You need call bootKernel() in test case setUp.

[dgafka]

@m2mtech that's true. Converter is needed for Symfony's UuidV7.
I am not sure why it did worked before

[m2mtech]

Could you please explain how this would be a Symfony bootstrapping issue? The same test setup works fine with Ecotone v1.255. Simply upgrading to v1.256—without changing anything else—causes the error. Also, the error message is identical in both the minimal test case and the full application.

[dgafka]

@unixslayer any thought if this can be #481 related?
or @jlabedo #478 ?

I do think there should be Converter, but if that worked before maybe that's indicator of some other bug

[m2mtech]

I'm aware that event-sourced events need to be stable over time and are serialized to JSON, so I’ve kept them limited to simple data types. For commands (used in CQRS), which run immediately—even if async—I was less strict and sometimes passed full entities. This worked fine until now, so I assumed it was acceptable.

Could it be that with v1.256, commands are now being serialized too, triggering this error? Adding a converter for Uuid isn’t an issue, but I’m concerned this change might cause similar serialization errors elsewhere. I doubt I’m the only one affected. If this is indeed a change in behavior, it might be more appropriate for a major version release rather than a minor one.

Also, I want to say I really appreciate your work and the immediate responses—thank you for the great support!

[dgafka]

@m2mtech I don't know context of your project, but I would say that sending an Entity via Command is odd, yet it should still work.
The problem that you face now happens because we want to serialize Command from Object to Array to retrieve identifier (AggregateIdentifierRetrievingService). However Aggregate Identifier should be retrievable from the Command itself without serialization. So from a quick look, it looks like there is some behaviour change.

[unixslayer]

@dgafka As this issue is not existing on 1.255.0 but in 1.256.0 it may be related to #479 and #482 however, I don't see any changes related to serialization. Maybe @jlabedo can give some more context to that. From my perspective, there was no behavior change in Ecotone in a manner of API changes therefore, this was not a candidate for major release.

I noticed, that serializer was released two weeks before. Maybe problem lies there?

The given example is a bit messy as it uses casting for Symfony\Component\Uid\Uuid to string in some places. Although, problem is not in retrieving aggregate id but rather serializing command itself as ChangeProductCommand->product->productId is typed with Symfony\Component\Uid\Uuid. When changing it to string, all works fine. This indicates serialization problem.

@m2mtech I didn't encounter that kind of issues after upgrading although, we have converters for every type that is not simple data type.

[m2mtech]

@unixslayer I'm using string identifiers for the aggregate (hence the casting). The UUID that's causing the issue is actually embedded within the command's payload object, not directly used as the aggregate identifier.

[m2mtech]

@unixslayer I also doubt that the serializer package is causing the problem, as the error disappears when you downgrade Ecotone to v1.255.0.

[unixslayer]

@unixslayer I'm using string identifiers for the aggregate (hence the casting). The UUID that's causing the issue is actually embedded within the command's payload object, not directly used as the aggregate identifier.

if App\Entity\Product is changed as follow

<?php

namespace App\Entity;

use Symfony\Component\Uid\Uuid;

class Product
{
    private ?string $productId = null;
    private ?string $name = null;

    public function getProductId(): ?Uuid
    {
        return $this->productId;
    }

    public function setProductId(?string $productId): static
    {
        $this->productId = $productId;

        return $this;
    }

    public function getName(): ?string
    {
        return $this->name;
    }

    public function setName(?string $name): static
    {
        $this->name = $name;

        return $this;
    }
}

test passes.