добавил логгер + добавил 1) headers для запроса в ройстат 2) явно конвертнул объект питона в json, как в документации к api

remotes/origin/roistat-logger-2-gleb
Gleb Mikhaylov 6 years ago
parent b103861001
commit f37e10138f
  1. 21
      apps/payment/tasks.py
  2. 5
      project/views.py

@ -1,5 +1,6 @@
import logging import logging
import requests import requests
import json
from mixpanel import Mixpanel from mixpanel import Mixpanel
@ -8,6 +9,7 @@ from django.conf import settings
from project.celery import app from project.celery import app
logger = logging.getLogger(__name__) logger = logging.getLogger(__name__)
logger_roistat = logging.getLogger('roistat')
@app.task @app.task
@ -53,13 +55,24 @@ def transaction_to_roistat(user_id, payment_id, event_name, amount, time, status
}] }]
project = settings.ROISTAT_PROJECT project = settings.ROISTAT_PROJECT
key = settings.ROISTAT_KEY key = settings.ROISTAT_KEY
url = settings.ROISTAT_API_URL + f'/project/add-orders?key={key}&project={project}' url = settings.ROISTAT_API_URL + \
resp = requests.post(url, json=body) f'/project/add-orders?key={key}&project={project}'
payload = json.dumps(body)
headers = {'content-type': 'application/json'}
resp = requests.request("POST", url, data=payload, headers=headers)
try: try:
resp_json = resp.json() resp_json = resp.json()
except: except:
resp_json = None resp_json = None
if resp.status_code != 200 or not resp_json or not resp_json.get('processed'): if resp.status_code != 200 or not resp_json or not resp_json.get('processed'):
logger.error('TRANSACTION_TO_ROISTAT for payment # %d, %s RUB: %s' % (payment_id, amount, resp.text)) logger.error('TRANSACTION_TO_ROISTAT for payment # %d, %s RUB: %s' % (
payment_id, amount, resp.text))
roistat_logger.debug('TRANSACTION_TO_ROISTAT for payment # %d, %s RUB: %s' % (
payment_id, amount, resp.text))
else: else:
logger.info('TRANSACTION_TO_ROISTAT: ' + str(resp)) logger.info('TRANSACTION_TO_ROISTAT: ' + str(resp.text))
roistat_logger.debug('TRANSACTION_TO_ROISTAT: ' + str(resp.text))

@ -12,9 +12,6 @@ from apps.course.models import Course
from apps.school.models import SchoolSchedule from apps.school.models import SchoolSchedule
from apps.payment.models import SchoolPayment, UserGiftCertificate, Payment, DrawingCampPayment from apps.payment.models import SchoolPayment, UserGiftCertificate, Payment, DrawingCampPayment
from apps.content.models import Banner, Package from apps.content.models import Banner, Package
import logging
logger_roistat = logging.getLogger('roistat')
User = get_user_model() User = get_user_model()
@ -139,8 +136,6 @@ class IndexView(TemplateView):
}) })
logger_roistat.debug('hi from home page!')
return context return context

Loading…
Cancel
Save