grandnode / grandnode2

Open-Source eCommerce Platform on .NET Core, MongoDB, AWS DocumentDB, Azure CosmosDB, LiteDB & Vue.js
https://grandnode.com/
GNU General Public License v3.0
1.06k stars 435 forks source link

Replace Custom Logger with System Logger Provider Implementation #439

Closed KrzysztofPajak closed 9 months ago

Nikhil13x commented 9 months ago

@KrzysztofPajak Is this PR removing the LOG view feature in the admin portal? Is it going to be added back in future?

KrzysztofPajak commented 9 months ago

@Nikhil13x In the context of ASP.NET Core application architecture, incorporating the separation of the login service from the main application database not only aligns with best practices but also allows for the effective use of advanced tools like Application Insights, which is available for free. Application Insights, a feature of Azure Monitor, is particularly beneficial in ASP.NET Core applications for monitoring the health, performance, and usage of web applications. By separating the login service, you can specifically monitor the authentication flow, identify bottlenecks, or detect anomalies like repeated failed login attempts, which could indicate a security threat. Application Insights provides detailed telemetry, including performance metrics, error logging, and usage trends. When used with a separate login service, it can give you a clearer picture of how the authentication process impacts overall application performance and user experience. Our next step will be remove Activity Log and miniprofiler. We also add new feature, integration with prometheus-net.