Skip to content

Commit

Permalink
Merge pull request #3299 from DFE-Digital/remove-application-insights
Browse files Browse the repository at this point in the history
Remove application insights
  • Loading branch information
martyn-w authored Oct 31, 2024
2 parents 0e05968 + 88bea80 commit 6fdefa9
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 48 deletions.
1 change: 0 additions & 1 deletion Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,6 @@ gem 'rack-attack'
gem 'rack-rewrite'
gem 'rack-timeout'

gem 'application_insights', github: 'microsoft/ApplicationInsights-Ruby', ref: 'a7429200'
gem 'openid_connect'
gem 'uk_postcode'

Expand Down
8 changes: 0 additions & 8 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -45,13 +45,6 @@ GIT
faraday_middleware-circuit_breaker
get_into_teaching_api_client

GIT
remote: https://github.com/microsoft/ApplicationInsights-Ruby.git
revision: a74292001cf12f6de957dfdcefaf8dbe415d7181
ref: a7429200
specs:
application_insights (0.5.7)

GIT
remote: https://github.com/mitsuru/validates_timeliness.git
revision: f28a62589cc069d1d1f5dfd99e1749edca04cd68
Expand Down Expand Up @@ -732,7 +725,6 @@ DEPENDENCIES
acts_as_list
addressable
amazing_print
application_insights!
bootsnap (>= 1.1.0)
brakeman (>= 6.0.1)
breasal
Expand Down
3 changes: 1 addition & 2 deletions app/controllers/concerns/maps_content_security_policy.rb
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,7 @@ module MapsContentSecurityPolicy
"https://www.google.com",
"https://maps.googleapis.com",
"https://www.googletagmanager.com",
"https://www.google-analytics.com",
"https://az416426.vo.msecnd.net" # needed for AppInsights
"https://www.google-analytics.com"
end
end
end
32 changes: 0 additions & 32 deletions config/initializers/application_insights.rb

This file was deleted.

1 change: 0 additions & 1 deletion config/initializers/content_security_policy.rb
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,6 @@
"https://www.google-analytics.com",
"https://maps.googleapis.com",
"googleapis.com",
"https://az416426.vo.msecnd.net", # needed for App Insights
"https://code.jquery.com/jquery-3.2.1.slim.min.js", # needed for Flipper UI
"https://cdnjs.cloudflare.com/ajax/libs/popper.js/1.12.9/umd/popper.min.js", # needed for Flipper UI
"https://maxcdn.bootstrapcdn.com/bootstrap/4.0.0/js/bootstrap.min.js" # needed for Flipper UI
Expand Down
4 changes: 0 additions & 4 deletions docs/documents/env-vars.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,6 @@ If required Exceptions Notifications can be sent to Sentry.

`SENTRY_DSN` - Send exception reports to sentry, config value supplied by Sentry. An organisation owner or #digital-tools-support can add new members to the school-experience team in the org dfe-teacher-services. Then a new key can be generated via https://sentry.io/settings/dfe-teacher-services/projects/school-experience/keys/.

## Monitoring

`APP_INSIGHTS_INSTRUMENTATION_KEY` - value supplied by Microsoft Application Insights

## Gitis configuration

`CRM_PRIVACY_CONSENT_ID` - GUID for the privacy policy consent id used by candidates - supplied by Gitis team
Expand Down

0 comments on commit 6fdefa9

Please sign in to comment.