-
Notifications
You must be signed in to change notification settings - Fork 0
NOEXP-181 zaimplementować i ustawić sentry serilog #17
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
NOEXP-181 zaimplementować i ustawić sentry serilog #17
The head ref may contain hidden characters: "NOEXP-181-Zaimplementowa\u0107-i-ustawi\u0107-Sentry-Serilog"
Conversation
Add configuration for logging
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements comprehensive logging and error monitoring by integrating Serilog and Sentry into the NoExp application. The implementation includes proper configuration across multiple environments, helper utilities for performance tracking, enhanced error handling in UI components, and extensive documentation to guide developers in using these tools effectively.
Key Changes:
- Integrated Serilog for structured logging with console and file sinks, including environment-specific configurations
- Integrated Sentry for error tracking and performance monitoring with proper security considerations
- Added helper classes for performance measurement and global error handling middleware
Reviewed changes
Copilot reviewed 12 out of 13 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| NoExp.Presentation/Program.cs | Bootstraps Serilog, configures Sentry integration, adds request logging and tracing middleware |
| NoExp.Presentation/NoExp.Presentation.csproj | Adds 8 NuGet packages for Serilog and Sentry functionality |
| NoExp.Presentation/appsettings.json | Configures base Serilog and Sentry settings with structured logging templates |
| NoExp.Presentation/appsettings.Development.json | Provides verbose logging configuration for development environment |
| NoExp.Presentation/appsettings.Production.json | Optimizes logging levels and retention for production deployment |
| NoExp.Presentation/Helpers/PerformanceLogger.cs | Implements disposable helper class for measuring and logging operation performance |
| NoExp.Presentation/Middleware/GlobalErrorHandlingMiddleware.cs | Creates middleware for global error handling (currently unused) |
| NoExp.Presentation/Components/Pages/Error.razor | Enhances error page with Sentry event ID display and automatic error logging |
| NoExp.Presentation/Components/Account/Pages/Register.razor | Adds comprehensive structured logging throughout registration flow with performance tracking |
| NoExp.Presentation/LOGGING_DOCUMENTATION.md | Provides complete documentation for Serilog and Sentry configuration and usage |
| NoExp.Presentation/LOGGING_QUICKSTART.md | Offers quick-start guide for setting up logging and monitoring |
| IMPLEMENTATION_SUMMARY.md | Summarizes all implementation details and next steps |
| .gitignore | Adds patterns to exclude log files and Sentry configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
twojnarowski
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Chyba na razie tyle
…ormance monitoring, structured logging (NOEXP-181)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 14 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 10 out of 11 changed files in this pull request and generated 14 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 9 out of 10 changed files in this pull request and generated 11 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…wić-Sentry-Serilog' into NOEXP-181-Zaimplementować-i-ustawić-Sentry-Serilog
fix comments
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 9 changed files in this pull request and generated 10 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This pull request introduces comprehensive logging and error monitoring improvements by integrating Serilog and Sentry across the application. The changes include new documentation, configuration updates, helper classes, and enhanced logging in key components. These updates provide structured logging, centralized error tracking, and performance monitoring, making debugging and maintenance easier and more robust.
Logging and Monitoring Integration
IMPLEMENTATION_SUMMARY.md, outlining all steps, configuration files, helper classes, and next actions for Serilog and Sentry integration.LOGGING_DOCUMENTATION.md) and a quickstart guide for configuring and using Serilog and Sentry, including best practices and troubleshooting.Codebase Enhancements
PerformanceLoggerhelper class and extension methods inNoExp.Presentation.Helpers, enabling easy measurement and logging of operation performance.Register.razorto use structured logging throughout the registration process, including debug, information, and warning logs, and integrated performance measurement for user registration. [1] [2] [3] [4] [5] [6] [7] [8]Error Handling Improvements
Error.razorto display Sentry Event IDs, automatically log unhandled exceptions, and integrate with the Sentry hub for error tracking. [1] [2] [3]These changes collectively provide a robust foundation for application logging, error monitoring, and performance tracking, greatly improving maintainability and observability.