8.3.8 (2024-02-23)

Overview of merged pull requests

BUGFIX: Dont log stack trace for `InvalidHashException in Production <https://github.com/neos/flow-development-collection/pull/3247>`_

This configures an invalidHashExceptions exception handler rendering group and configures it to not dump stack traces in Production context. For Development context stack traces are still written to ease debugging.

See: #3159

Upgrade instructions

In case you need trace dumps for InvalidHashException in production context, override the settings as needed.

Review instructions

See: #3159 for ways to trigger those exceptions. Then check if a trace is dumped.
Packages: Flow

BUGFIX: Use configured cache factory 

The previously used method to select the configured CacheFactory class from the Objects.yaml never returned a result as the raw configuration split by package name was accessed, therefore the fallback was used and it was not possible to provide a custom CacheFactory.

Resolves: #3317
Packages: Flow

BUGFIX: Normalize source type for converting floats to ValueObjects 

This change allows the ScalarTypeToObjectConverter to convert from floats to ValueObjects. The type converter uses gettype() which will return double for floats. As the constructor argument type of the ValueObject is passed through TypeHandling::normalizeType(), it will always be float and never double which will lead to floats not being converted by this type converter.

Packages: Flow

BUGFIX: Web-Exception messages preserve whitespace (ASCII Art)

Closes #2696

Exceptions which have multiple spaces in their $message where previously not as expected displayed. Exceptions outputted to the CLI or logged did preserve multiple whitespaces naturally, but since they are collapsed in HTML by default, they are not shown in the browser.

[x] Code follows the PSR-2 coding style
[ ] Tests have been created, run and adjusted as needed
[x] The PR is created against the lowest maintained branch

Packages: Flow

BUGFIX: More descriptive reflection error if attribute does not exist 

Adds error catching and more context information if an attribute could not be instantiated.

Resolves: #3178

Upgrade instructions

Packages: Flow

TASK: Followup 8.3 Revert “BUGFIX: Sanitize uploaded svg files for suspicious contents”

Followup #3249

This reverts commit https://github.com/neos/flow-development-collection/commit/a1642ef31f19a974f34a302c98c13c77b6422ba1 fully for 8.3

In the upmerge https://github.com/neos/flow-development-collection/commit/`e2b895120eb00a6c2a352ce22d84f0302b6c3c71 <https://github.com/neos/flow-development-collection/commit/e2b895120eb00a6c2a352ce22d84f0302b6c3c71>``_#diff-``b54379d1fdcdc7a8433b179ba5a080f729e68a55 <https://github.com/neos/flow-development-collection/commit/b54379d1fdcdc7a8433b179ba5a080f729e68a55>`_9cd2a6203e9d1c49b7789c3e this composer json adjustment in Flow got lost.

Packages: Flow

BUGFIX: Fix documentation of `maximumThrowableDump defaults <https://github.com/neos/flow-development-collection/pull/3269>`_

Related: #3158
Packages: Flow

TASK: ObjectManagement limit debugBacktrace 

debug_backtrace is expensive, and we are only interested in the second callee.

Packages: Flow

TASK: Scripts.php avoid use of same references for exec 

Followup to https://github.com/neos/flow-development-collection/pull/3116

I debugged with @dlubitz a problem and we found that in theory? $output _might_ be possibly already filled? I dont know if that can happen but just to be sure we set it to empty as its also a bad practice.

Upgrade instructions

Packages: Flow

TASK: Fix errors in Middleware code example 

I found two small errors in the Middleware code example in the documentation.

Packages: Flow

TASK: Correct doc types StringFrontend::get 

… and document FrontendInterface::get correctly

This is required for the phpstan migration in Neos Cms ;)

Packages: Flow Cache

TASK: Adjust phpstan command to Neos’ dev collection 

There we also use composer run lint:phpstan and composer run lint

Packages: Flow

TASK: Phpstan level 2 for Flow 8.3 and ignore to be fixed things 

Flow 9 Pr https://github.com/neos/flow-development-collection/pull/3217

Upgrade instructions

Packages: .github Eel Flow FluidAdaptor

TASK: Correct symfony dependencies in Flow composer.json 

The upmerge commit e2b895120eb00a6c2a352ce22d84f0302b6c3c71 <https://github.com/neos/flow-development-collection/commit/e2b895120eb00a6c2a352ce22d84f0302b6c3c71>``_ wrongly removed symfony 6.0 in the version constraints of neos/flow.

Relates: #2999
Packages: Flow

TASK: Correct Flow composer.json 

The upmerge commit 42e3fd7886d5bed317511a2046d4119867216923 wrongly overwrote major parts of Flows composer.json, introducing older versions of PHP and psr/log as well as removing dependencies on some other psr packages. This change corrects the issue and needs to be upmerged accordingly.

The changes were never merged into the collection composer.json so that the issue was not noticed in development environments.

Packages: Flow