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

callback.php kan niet aangeroepen worden #14

Closed
bgoedhart opened this issue Sep 21, 2016 · 13 comments
Closed

callback.php kan niet aangeroepen worden #14

bgoedhart opened this issue Sep 21, 2016 · 13 comments
Assignees
Labels
Milestone

Comments

@bgoedhart
Copy link

Hi,

Wij zijn bezig met de Mollie plugin te testen in WHMCS. De betaling zelf gaat goed, maar Mollie kan de callback.php niet bereiken. In de apache logfile zie je alleen maar een error 500. We komen er maar niet achter waarom callback.php niet bereikbaar is van buitenaf. Het bestand bestaat gewoon op de server. mollie.php is wel bereikbaar in dezelfde directory. We hebben geen .htaccess bestanden staan die het eventueel zouden kunnen verpesten. Enig idee, wat hier mis gaat?

@ghost
Copy link

ghost commented Sep 21, 2016

Goedemorgen,

Ik weet dat dit heeft te maken dat mollie verbinding wil maken met de callback.php via een bepaalde poort.
Alleen weet ik op dit moment niet meer welke dat is:-(

Met vriendelijk groet,
André

@bgoedhart
Copy link
Author

Hoi André,

Maar dan zou ik het bestand toch zelf wel moeten kunnen aanroepen via HTTPS zonder een 500 error te krijgen?

Met vriendelijke groet,

Benno Goedhart

@ghost
Copy link

ghost commented Sep 21, 2016

Beste,

Ik heb dit even getest in mijn whmcs en via de rechtstreekse url is deze niet aan te roepen.

Met vriendelijk groet André

@bgoedhart
Copy link
Author

Ok dan doe ik nog even een test betaling en wacht af of Mollie terug komt met een foutmelding.

@Wouter0100
Copy link
Member

Wouter0100 commented Sep 21, 2016

Howdy! Zouden jullie even error logging aan willen zetten en kijken wat er in de errors komt, als Mollie de callback.php probeert aan te roepen?

Kunnen jullie doen door even op een paar plekken in de callback.php de volgende regels te plaatsen:

ini_set('error_reporting', E_ALL & ~E_NOTICE & ~E_DEPRECATED);
ini_set('log_errors', 1);

Ik vraag jullie expres om dit op een paar plekken te doen, omdat WHMCS dit blijkbaar overschrijft. Grr. Bij voorkeur dus even onder regel 6 en nogmaals bij regel 12.

@Wouter0100 Wouter0100 self-assigned this Sep 21, 2016
@Wouter0100 Wouter0100 added this to the v1.1 milestone Sep 21, 2016
@Wouter0100 Wouter0100 added the bug label Sep 21, 2016
@ghost
Copy link

ghost commented Sep 21, 2016

Als dit de goede url: http://klanten.thehosting4you.nl/modules/gateways/mollie/callback.php is krijg ik deze melding: De pagina op klanten.thehosting4you.nl werkt niet
klanten.thehosting4you.nl kan dit verzoek momenteel niet verwerken.
HTTP ERROR 500

Met vriendelijk groet André

@Wouter0100
Copy link
Member

De URL moet je ook niet handmatig aanroepen, maar via een testbetaling. Handmatig aanroepen geeft hij inderdaad een 500 terug. Zie https://github.com/0100Dev/WHMCS-Mollie/blob/cb6ec43159b1e59c0c277cf378cb6d3de45232ab/src/mollie/callback.php#L21-L39.

@ghost
Copy link

ghost commented Sep 22, 2016

De URL moet je ook niet handmatig aanroepen, maar via een testbetaling. Handmatig aanroepen geeft hij inderdaad een 500 terug. Zie https://github.com/0100Dev/WHMCS-Mollie/blob/cb6ec43159b1e59c0c277cf378cb6d3de45232ab/src/mollie/callback.php#L21-L39.

Er zijn vandaag klanten bij mij geweest die betaling hebben gedaan zonder problemen.
de facturen worden netjes op betaald gezet en teven op de factuur de transactie id.

Ik hoop dat dit voldoende is.

@Wouter0100
Copy link
Member

Dan was het probleem dus niet aan onze kant. Is het bij starter ook opgelost, @bgoedhart ?

@miroderuiter
Copy link

Ik ben een collega van Benno Goedhart. We hebben in het bestand callback.php de regels header('HTTP/1.1 500 etc.) uitgecommentarieerd. Nu lijkt de betaling wel goed te gaan. We krijgen van Mollie ook geen foutmeldingen terug.

@Wouter0100
Copy link
Member

@miroderuiter Heb je ook de exit(); eruit gehaald? Deze is namelijk echt required, en als dat header weghalen effect had, was er iets fout met de callback vanuit Mollie (of de betaling was al verwerkt, of hij bestond niet).

Gaf Mollie die foutmelding terug, of hadden jullie het zelf getest wat die error gaf?

@miroderuiter
Copy link

Nee, de exit(); heb ik laten staan. Mollie gaf geen foutmelding terug. We wilden eigenlijk door het verwijderen van de header() regel kijken of er een foutmelding in beeld kwam, maar heel verassend ging het ineens goed.

@Wouter0100
Copy link
Member

Hmm, vreemd. Dan komt het aanroepen van de callback in een bepaald scenario terecht waar hij niet in zou moeten komen. Daarom sturen we dus ook een error, wat Mollie mooi oppakt.

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

3 participants