lucaspalomodevelop/indico-plugins
1
0
Fork 0
mirror of https://github.com/lucaspalomodevelop/indico-plugins.git synced 2026-03-13 07:29:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

Payment/PayPal: Replace urllib with requests

Browse Source
This commit is contained in:
Ilias Trichopoulos 2015-01-09 11:48:58 +01:00 committed by Adrian Moennich
parent 38a8a67115
commit 5b52d3edbb
1 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
payment_paypal/indico_payment_paypal/controllers.py
View File

@ -17,12 +17,11 @@
from __future__ import unicode_literals
from itertools import chain
from urllib import urlencode
from urllib2 import urlopen
from werkzeug.exceptions import BadRequest
import requests
from flask import request
from flask_pluginengine import current_plugin
from werkzeug.exceptions import BadRequest
from MaKaC.conference import ConferenceHolder
from MaKaC.webinterface.rh.base import RH
@ -53,12 +52,10 @@ class RHPaymentEventNotify(RH):
def _process(self):
if not self._verify_business() or not self._verify_amount():
return
verify_params = chain(IPN_VERIFY_EXTRA_PARAMS, request.form.iteritems())
verify_string = urlencode(list(verify_params))
response = urlopen(current_plugin.settings.get('url'), data=verify_string)
result = response.read()
verify_params = list(chain(IPN_VERIFY_EXTRA_PARAMS, request.form.iteritems()))
result = requests.post(current_plugin.settings.get('url'), data=verify_params).text
if result != 'VERIFIED':
current_plugin.logger.warning("Paypal IPN string {} did not validate ({})".format(verify_string, result))
current_plugin.logger.warning("Paypal IPN string {} did not validate ({})".format(verify_params, result))
return
if self._is_transaction_duplicated():
current_plugin.logger.info("Payment not recorded because transaction was duplicated\n"