View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0003929 | OXID eShop (all versions) | 1. ----- eShop frontend ----- | public | 2012-04-25 20:30 | 2012-12-10 14:16 |
Reporter | Adrian.Kirchner | Assigned To | |||
Priority | urgent | Severity | block | Reproducibility | always |
Status | resolved | Resolution | unable to reproduce | ||
Product Version | 4.5.9 revision 43186 | ||||
Summary | 0003929: [EE-only] Mall ssl url detection is broken | ||||
Description | When adding a subshop with ssl url, oxid won't identify this mall by the assigned ssl url. This happens because oxid core developers introduced a new regex based url parser in oxConfig::isCurrentUrl() which ignores the https scheme. | ||||
Steps To Reproduce | 1) setup a fresh EE instance (eg. V4.5.9) 2) add a subshop with ssl url 3) visit subshop by ssl url -> won't work | ||||
Additional Information | Because this bugtracker is publicly available, I diffed the attached patch against the public trunk[1]. Have you considered the use of phps native parse_url-function[2] for this purpose? It should be much more reliable and provides all information you need at that point. [1] http://svn.oxid-esales.com/trunk/eshop/core/oxconfig.php [2] http://php.net/manual/en/function.parse-url.php | ||||
Tags | Subshops | ||||
Attached Files | oxconfig.php.patch.txt (762 bytes)
--- oxconfig.php-revBASE.svn000.tmp.php Fr Mrz 23 07:14:31 2012 +++ oxconfig.php Mi Apr 25 20:01:57 2012 @@ -784,10 +784,10 @@ $oUtilsServer = oxUtilsServer::getInstance(); - preg_match("/^(http:\/\/)?([^\/]+)/i", $sURL, $matches); + preg_match("/^(http[s]?:\/\/)?([^\/]+)/i", $sURL, $matches); $sUrlHost = $matches[2]; - preg_match("/^(http:\/\/)?([^\/]+)/i", $oUtilsServer->getServerVar( 'HTTP_HOST' ), $matches); + preg_match("/^(http[s]?:\/\/)?([^\/]+)/i", $oUtilsServer->getServerVar( 'HTTP_HOST' ), $matches); $sRealHost = $matches[2]; $sCurrentHost = preg_replace( '/\/\w*\.php.*/', '', $oUtilsServer->getServerVar( 'HTTP_HOST' ) . $oUtilsServer->getServerVar( 'SCRIPT_NAME' ) ); | ||||
Theme | Both | ||||
Browser | All | ||||
PHP Version | any | ||||
Database Version | any | ||||
related to | 0004010 | resolved | vilma_liorensaityte | force_sid added in https to every link |
related to | 0004026 | resolved | vaidas.matulevicius | oxConfig::isSsl() doesn't work if only sAdminSSLURL is activated |