Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace Custom Logger with System Logger Provider Implementation #439

Merged
merged 14 commits into from
Nov 30, 2023

Conversation

KrzysztofPajak
Copy link
Member

No description provided.

@KrzysztofPajak KrzysztofPajak merged commit 31cb21d into main Nov 30, 2023
@Nikhil13x
Copy link
Contributor

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

@KrzysztofPajak
Copy link
Member Author

@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.

@KrzysztofPajak KrzysztofPajak deleted the feature/logger-provider branch March 3, 2024 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants