Skip to content

Commit

Permalink
Delete old header bidding endpoint (#27678)
Browse files Browse the repository at this point in the history
  • Loading branch information
Jakeii authored Dec 19, 2024
1 parent 0537940 commit a3726fd
Show file tree
Hide file tree
Showing 6 changed files with 2 additions and 27 deletions.
1 change: 0 additions & 1 deletion commercial/app/controllers/CommercialControllers.scala
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ trait CommercialControllers {
lazy val piggybackPixelController = wire[PiggybackPixelController]
lazy val cmpDataController = wire[CmpDataController]
lazy val adsDotTextFileController = wire[AdsDotTextViewController]
lazy val prebidAnalyticsController = wire[PrebidAnalyticsController]
lazy val passbackController = wire[PassbackController]
lazy val ampIframeHtmlController = wire[AmpIframeHtmlController]
lazy val nonRefreshableLineItemsController = wire[nonRefreshableLineItemsController]
Expand Down
18 changes: 0 additions & 18 deletions commercial/app/controllers/PrebidAnalyticsController.scala

This file was deleted.

3 changes: 0 additions & 3 deletions commercial/conf/routes
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,6 @@ GET /commercial/cmp/shortvendorlist.json
GET /ads.txt commercial.controllers.AdsDotTextViewController.renderTextFile()
GET /app-ads.txt commercial.controllers.AdsDotTextViewController.renderAppTextFile()

# Analytics
POST /commercial/api/hb commercial.controllers.PrebidAnalyticsController.insert()

# Passbacks
GET /commercial/ias-passback/:size commercial.controllers.PassbackController.renderIasPassback(size)

Expand Down
2 changes: 0 additions & 2 deletions common/app/common/configuration.scala
Original file line number Diff line number Diff line change
Expand Up @@ -515,8 +515,6 @@ class GuardianConfiguration extends GuLogging {

lazy val expiredPaidContentUrl = s"${site.host}/info/2015/feb/06/paid-content-removal-policy"

lazy val prebidAnalyticsStream = configuration.getMandatoryStringProperty("commercial.prebid.analytics.stream")

lazy val prebidServerUrl =
configuration.getStringProperty("commercial.prebid.server.url") getOrElse "http://localhost:8000"

Expand Down
4 changes: 2 additions & 2 deletions common/app/http/RequestLoggingFilter.scala
Original file line number Diff line number Diff line change
Expand Up @@ -29,8 +29,8 @@ class RequestLoggingFilter(implicit val mat: Materializer, executionContext: Exe
case _ => ""
}
}
// don't log uncacheable /commercial/api/hb POST requests due to the volume of them
if (rh.method != "POST" || rh.path != "/commercial/api/hb") {
// don't log uncacheable POST requests due to the volume of them
if (rh.method != "POST") {
requestLogger.withResponse(response).info(s"${rh.method} ${rh.uri}$additionalInfo")
}
case Failure(error) =>
Expand Down
1 change: 0 additions & 1 deletion dev-build/conf/routes
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,6 @@ GET /commercial/non-refreshable-line-items.json
GET /ads.txt commercial.controllers.AdsDotTextViewController.renderTextFile()
GET /app-ads.txt commercial.controllers.AdsDotTextViewController.renderAppTextFile()
GET /commercial/ias-passback/:size commercial.controllers.PassbackController.renderIasPassback(size)
POST /commercial/api/hb commercial.controllers.PrebidAnalyticsController.insert()

# Commercial Admin
GET /commercial controllers.admin.CommercialController.renderCommercialMenu()
Expand Down

0 comments on commit a3726fd

Please sign in to comment.