Skip to content

Commit

Permalink
Update sentry sdk (#474)
Browse files Browse the repository at this point in the history
* Update sentry sdk

* Replace capture_message with logging

---------

Co-authored-by: Kartik Ohri <kartikohri13@gmail.com>
  • Loading branch information
mayhem and amCap1712 authored Jul 23, 2024
1 parent 0be8f6d commit 5795778
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions metabrainz/model/payment.py
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@
from metabrainz.payments import Currency, SUPPORTED_CURRENCIES
from metabrainz.payments.receipts import send_receipt
from metabrainz.admin import AdminModelView
from sentry_sdk import capture_message
from sqlalchemy.sql import func, desc
from flask import current_app
from datetime import datetime
Expand Down Expand Up @@ -173,7 +172,7 @@ def process_paypal_ipn(cls, form):
if form['payment_status'] != 'Completed':
# TODO(roman): Convert to regular `logging.info` call when such detailed logs
# are no longer necessary to capture.
capture_message("PayPal: Payment is not completed", level="info", extra={"ipn_content": form})
logging.info("PayPal: Payment is not completed: %s",form)
return

account_ids = current_app.config['PAYPAL_ACCOUNT_IDS'] # "currency => account" mapping
Expand Down
2 changes: 1 addition & 1 deletion requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ rauth==0.7.3
redis==5.0.3
reportlab==4.1.0
requests==2.32.0
sentry-sdk[flask]==1.43.0
sentry-sdk[flask]==2.10.0
six==1.16.0
SQLAlchemy==1.4.52
sqlalchemy-dst>=1.0.1
Expand Down

0 comments on commit 5795778

Please sign in to comment.