Nicolas Grekas comments

Results 736 comments of


                                            Nicolas Grekas

trafficstars

[DependencyInjection] Fix ignore invalid_reference behavior param for the some services

@vtsykun did you check my comment? WDYT?

[Notifier] Support for desktop notifications via `jolicode/JoliNotif`

Are there any technical implications related to using Push or Desktop for channels? (Desktop looks fine to me if we need to go with another channel)

Deserialize array of scalars to its doctype

Help wanted Keeping this open won't make it happen magically ;)

[Mailer] Option to enforce STARTTLS

#53621 paved the way to show how this should be done. Could be by renaming auto_tls to just tls and support bool|auto as a value. PR welcome.

[WebProfilerBundle] clear or remove session

Friendly ping @Oipnet - up to move forward on this PR? I'm wondering if we shouldn't leverage the session service instead of using the global functions BTW.

[TypeInfo] Redesign Type methods and nullability

Thank you @mtarld.

[Serializer] Support subclasses of `DateTime` and `DateTimeImmutable`

Thank you @amcsi.

[Process] Also check `PATH` in `ExecutableFinder` if `open_basedir` is set

Thanks for the nice explanations. Checking the CI, the failure seems related, can you please have a look?

[Process] Also check `PATH` in `ExecutableFinder` if `open_basedir` is set

Closing in favor of #58291

"Cannot modify header information" error encountered while streaming

Thanks for the reproducer. I couldn't reproduce, so other conditions are required to see the issue I suppose. I ran the app using `symfony serve`, which should work like any...