Entity Framework Logging Behaviour

[statler]

Describe the bug

The docs describe the known issue with logging of the whole database as below:

WARNING: In older versions of Serilog.Exceptions, if you are using EntityFrameworkCore with Serilog.Exceptions you must add this, otherwise in certain cases your entire database will be logged! This is because the exceptions in Entity Framework Core have properties that link to the entire database schema in them (See #100, dotnet/efcore#15214). Newer versions of Serilog.Exceptions avoids this issue by preventing the destructure of properties that implement IQueryable preventing their execution.

In older versions of Serilog.Exceptions Can you please clarify what versions this is necessary for. 'Older versions' is not definitive or informative. Is it still a requirement or not.

Steps to reproduce

View the docs

Expected behaviour

The requirement for what versions the additional library is required for should be explicit, or at least definitive about whether the issue is present in the current build.

[RehanSaeed]

This was the PR that had the fix in it:

• Updates ReflectionBasedDestructurer to destructure IQueryable values to prevent enumeration. #425

This made it into the 8.0.0 release:

• https://github.com/RehanSaeed/Serilog.Exceptions/releases/tag/8.0.0

Documentation updated.

[statler]

Superb. Thank you for your work

…

On Tue, 7 Dec 2021, 9:38 pm Muhammad Rehan Saeed, ***@***.***> wrote: This was the PR that had the fix in it: - #425 <#425> This made it into the 8.0.0 release: - https://github.com/RehanSaeed/Serilog.Exceptions/releases/tag/8.0.0 Documentation updated. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#439 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACSQRYFYK3SLAC5KCGBXZTUPXP3ZANCNFSM5JMIYDJA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.