Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Paypal Endpunkte geändert #1657

Closed
baumannsven opened this issue Jun 3, 2016 · 6 comments
Closed

Paypal Endpunkte geändert #1657

baumannsven opened this issue Jun 3, 2016 · 6 comments
Labels
Milestone

Comments

@baumannsven
Copy link

Bei Paypal haben sich die Endpunkte geändert.

https://github.com/isotope/core/blob/master/system/modules/isotope/library/Isotope/Model/Payment/Paypal.php#L50

alte Zeile:
$objRequest->send(('https://www.' . ($this->debug ? 'sandbox.' : '') . 'paypal.com/cgi-bin/webscr?cmd=_notify-validate'), file_get_contents("php://input"), 'post');

neue Zeile:
$objRequest->send(('https://ipnpb.' . ($this->debug ? 'sandbox.' : '') . 'paypal.com/cgi-bin/webscr?cmd=_notify-validate'), file_get_contents("php://input"), 'post');

Beschreibung von Paypal:
Die Endpunkte ipnpb.paypal.com und ipnpb.sandbox.paypal.com akzeptieren nur HTTPS-Verbindungen. Wenn Sie zurzeit www.paypal.com benutzen, sollten Sie zu ipnpb.paypal.com wechseln, sobald Sie Ihren Code aktualisieren, um HTTPS zu nutzen.

Ist im Shop getestet worden und funktioniert.

@aschempp
Copy link
Member

aschempp commented Jun 3, 2016

Funktioniert aber dann nur über HTTPS, und die bestehenden auch über HTTP. Wenn wir das einfach ändern würden bestehende Shops ohne SSL nicht mehr funktionieren…?

@baumannsven
Copy link
Author

Dann würde ich das so lösen.

$subdomain = 'www';
if (protocol === 'https'){
$subdomain = 'ipnb';
}

$objRequest->send(('https://' . $subdomain . '.' . ($this->debug ? 'sandbox.' : '') . 'paypal.com/cgi-bin/webscr?cmd=_notify-validate'), file_get_contents("php://input"), 'post');

@baumannsven
Copy link
Author

Ich habe Paypal kontaktiert sobald ich näheres weiß was die IPN angeht werde ich es hier mitteilen

@baumannsven
Copy link
Author

Habe nun alle Informationen gesammelt was die Endpunkte von Paypal angehen.
Der ipnpb.paypal.com Endpunkt ist ausschließlich vor Shops mit SSL Zertifikat. Daher müsste man es wie oben in der vorgeschlagenen If Schleife realisieren.

@aschempp

@aschempp
Copy link
Member

Funktioniert denn TLS nicht mit der WWW-Adresse? Weil nach meinem Wissen nutzen das sicherlich sehr viele bereits jetzt schon?

Ausserdem kommt bezüglich PayPal bzw. allgemein Anforderungen einiges auf uns zu: https://www.paypal-knowledge.com/infocenter/index?page=content&widgetview=true&id=FAQ1913&viewlocale=de_DE

@aschempp aschempp added the bug label Sep 2, 2016
@aschempp aschempp added this to the 2.4.0 milestone Sep 2, 2016
@aschempp
Copy link
Member

aschempp commented Sep 3, 2016

Ich habe mich nochmals mit dem Thema beschäftigt und festgestellt dass wir gar kein Problem haben, da wir schon immer nur HTTPS verwenden. Ich dachte erst es ist abhängig von der Webseite 😎

Die URL ist in Isotope 2.4 entsprechend angepasst. Relevante Infos: https://www.paypal-knowledge.com/infocenter/index?page=content&widgetview=true&id=FAQ1916&viewlocale=de_DE

@aschempp aschempp closed this as completed Sep 3, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants