neos / flow-development-collection

The unified repository containing the Flow core packages, used for Flow development.
https://flow.neos.io/
MIT License
134 stars 187 forks source link

Duplicate inclusion of security context hash in different places #3373

Open kdambekalns opened 2 days ago

kdambekalns commented 2 days ago

https://github.com/neos/flow-development-collection/commit/5fd0550ee489b32ab1c6d76a0b10eb0cb839f575#diff-972b73b496f48b9644bd8e60c8e2492dba1bfb7b6dc1bd49dc22ac316bc3cfe3

and

https://github.com/neos/flow-development-collection/commit/b4229a295b403dc3729c7d92f08e00c0f942314a

both add the security context hash in different places which should mean it is used doubly (as the parameters of the SqlFilter are added into the cache id Doctrine generates).

We should figure out why both are needed at some point.

Originally posted by @kitsunet in https://github.com/neos/flow-development-collection/issues/2637#issuecomment-2196881594