Skip to content

Commit 63f7e51

Browse files
author
Dale Sikkema
committed
MAGETWO-45159: [ZAP-M2]: Cookie set without secure flag
1 parent 180ca97 commit 63f7e51

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

app/code/Magento/Backend/Model/Session/AdminConfig.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ public function __construct(
8484
$adminPath = $this->extractAdminPath();
8585
$this->setCookiePath($adminPath);
8686
$this->setName($sessionName);
87+
$this->setCookieSecure($this->_httpRequest->isSecure());
8788
}
8889

8990
/**

app/code/Magento/Backend/Test/Unit/Model/Session/AdminConfigTest.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -105,12 +105,13 @@ public function testSetCookiePathNonDefault()
105105
}
106106

107107
/**
108-
* Test for setting session name for admin
109-
*
108+
* Test for setting session name and secure_cookie for admin
110109
*/
111-
public function testSetSessionNameByConstructor()
110+
public function testSetSessionSettingsByConstructor()
112111
{
113112
$sessionName = 'admin';
113+
$secureRequest = true;
114+
$this->requestMock->expects($this->once())->method('isSecure')->willReturn($secureRequest);
114115

115116
$validatorMock = $this->getMockBuilder('Magento\Framework\Validator\ValidatorInterface')
116117
->disableOriginalConstructor()
@@ -136,5 +137,6 @@ public function testSetSessionNameByConstructor()
136137
]
137138
);
138139
$this->assertSame($sessionName, $adminConfig->getName());
140+
$this->assertSame($secureRequest, $adminConfig->getCookieSecure());
139141
}
140142
}

0 commit comments

Comments
 (0)