View Issue Details

IDProjectCategoryView StatusLast Update
0006253module PayPalmodule PayPal - subpublic2015-11-02 17:09
Reporterrhuelsberg Assigned To 
PrioritynormalSeveritytrivialReproducibilityalways
Status closedResolutionfixed 
Product Version3.1.2 
Summary0006253: oePayPalIPNHandler::handleRequest redirect cl=1
DescriptionClass: oePayPalIPNHandler
Function: handleRequest

If the IPN verification returns "true", the funtction handleRequest return true and OXID will redirect the robot "PayPal IPN ( https://www.paypal.com/ipn )" to /index.php?cl=1
The Class "1" is invalid and OXID will redirect the robot to "cl=start".

Please update the return value of handleRequest or implement an output value like:

die( $blRequestHandled ? 'valid' : 'invalid' );
Additional InformationAccess Log
[...] "POST /index.php?cl=oePayPalIPNHandler&fnc=handleRequest&shp=2 HTTP/1.1" 302 - "-" "PayPal IPN ( https://www.paypal.com/ipn )"
[...] "POST /index.php?cl=1&sid=[...] HTTP/1.1" 302 - "-" "PayPal IPN ( https://www.paypal.com/ipn )"
[...] "POST /index.php?cl=start&redirected=1 HTTP/1.1" 200 575510 "-" "PayPal IPN ( https://www.paypal.com/ipn )"
TagsNo tags attached.
Attached Files
oepaypalipnhandler.txt (1,841 bytes)   
<?php
/**
 * This file is part of OXID eSales PayPal module.
 *
 * OXID eSales PayPal module is free software: you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * OXID eSales PayPal module is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with OXID eSales PayPal module.  If not, see <http://www.gnu.org/licenses/>.
 *
 * @link      http://www.oxid-esales.com
 * @copyright (C) OXID eSales AG 2003-2013
 */

/**
 * PayPal IPN handler class.
 *
 * Handle PayPal notifications.
 *  - Extract data.
 *  - Check if valid.
 *  - Call order methods to save data.
 */
class oePayPalIPNHandler extends oePayPalController
{

	// [...]

    /**
     * IPN handling function.
     *  - Call to check if request is valid (from PayPal and to correct shop).
     *  - Initiate payment status changes according to IPN information.
     *
     * @return null
     */
    public function handleRequest()
    {
        $blRequestHandled = false;
        $blRequestValid = $this->requestValid();

        if ( $blRequestValid ) {
            $oRequest = $this->getPayPalRequest();
            $oLang = $this->getPayPalConfig()->getLang();

            $oProcessor = $this->getProcessor();
            $oProcessor->setRequest( $oRequest );
            $oProcessor->setLang( $oLang );

            $blRequestHandled = $oProcessor->process();
        }
        die( $blRequestHandled ? 'valid' : 'invalid' );
    }

	// [...]
	
}
oepaypalipnhandler.txt (1,841 bytes)   

Relationships

duplicate of 0006132 resolvedanton.fedurtsya POST to /index.php?cl=1 caused by redirect in the answers to the IPN call back causes exceptions 

Activities

rhuelsberg

2015-11-02 15:56

reporter   ~0011294

Oh found another ticket to this problem: https://bugs.oxid-esales.com/view.php?id=6132

SRY, please close this ticket.

QA

2015-11-02 17:09

administrator   ~0011296

closed as duplicate of 0006132