Skip to content
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

Automatically send Rails logs to AppInsights - #5

Merged
denis-sokolov merged 4 commits into
FormulaMonks:masterfrom
c-w:rails-logging
Feb 14, 2020
Merged

denis-sokolov merged 4 commits into
FormulaMonks:masterfrom
c-w:rails-logging

Conversation

@c-w

@c-w c-w commented Dec 17, 2019 •

Copy link
Copy Markdown
Contributor

Currently this gem enables Rails applications to send telemetry such as requests and exceptions to Azure Application Insights, via ApplicationInsights::Rack::TrackRequest and AppInsights::ExceptionHandling respectively. To make it even easier to debug of Rails applications that use this gem, this pull request augments the existing functionality to also send logs to the Application Insights via the TrackTrace API. AppInsights plugins for other popular server frameworks implement similar functionality, e.g. refer to the plugins for Flask and Django.

To reduce the performance impact of turning on log shipping, this pull request also adds a configuration option to make the calls to the application_insights SDK non-blocking and have a background thread perform the HTTP requests to the Application Insights service.

@denis-sokolov
denis-sokolov merged commit e4ccf5d into FormulaMonks:master Feb 14, 2020
@denis-sokolov

Copy link
Copy Markdown
Contributor

Thank you for your contribution!

As you can see, we do not actively maintain this module at the moment, so apologies for the inconvenience.

@denis-sokolov

Copy link
Copy Markdown
Contributor

Released in 0.0.6.

@c-w

c-w commented Feb 14, 2020

Copy link
Copy Markdown
Contributor Author

Thanks for the merge! Although the gem isn't maintained, if does exactly what I need it to do, so thanks so much for making this available :)

@c-w
c-w deleted the rails-logging branch February 14, 2020 13:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants