Skip to content
This repository has been archived by the owner on Jan 31, 2018. It is now read-only.

Commit

Permalink
Merge pull request #603 from willkg/1173763-remove
Browse files Browse the repository at this point in the history
[bug 1173763] Remove logging from sumosuggest provider
  • Loading branch information
willkg committed Jun 16, 2015
2 parents 3ae5b91 + 1436f53 commit 9a0e8ec
Showing 1 changed file with 0 additions and 18 deletions.
18 changes: 0 additions & 18 deletions fjord/suggest/providers/sumosuggest.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
import requests

from fjord.base.google_utils import ga_track_event
from fjord.journal.utils import j_info
from fjord.suggest import Link, Suggester
from fjord.redirector import Redirector, build_redirect_url

Expand Down Expand Up @@ -123,23 +122,6 @@ def handle_redirect(self, request, redirect):
# This doc doesn't exist.
return

# FIXME: Putting this in here because I can't figure out
# what's going on with bug #1173763 and hoping this sheds some
# light. We'll take this out as soon as we identify the issue.
j_info(
app='suggest',
src='sumosuggest',
action='redirect',
msg='url is https:',
metadata={
'response_id': response_id,
'redirect': redirect,
'rank': rank,
'url': url,
'docs': docs
}
)

ga_track_event({
'cid': str(response_id),
'ec': EVENT_CATEGORY,
Expand Down

0 comments on commit 9a0e8ec

Please sign in to comment.