mirror
/
PHPMailer
mirror of https://github.com/PHPMailer/PHPMailer.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SecurityDenialOfServiceVectorsTest: various test tweaks 

Minor test tweaks:
* Add `@coversNothing` tag.
* Rename the first test (add a number).
* Use `@link` instead of `@see` for links.
* Minor comment punctuation.
This commit is contained in:
jrfnl 2021-06-24 19:10:04 +02:00
parent 6b95da3e65
commit eb6dca5521
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
test/Security/DenialOfServiceVectorsTest.php
View File

@ -17,6 +17,8 @@ use PHPMailer\Test\SendTestCase;
/**
* Test denial of service attack vectors, which have been mitigated.
*
* @coversNothing
*/
final class DenialOfServiceVectorsTest extends SendTestCase
{
@ -24,9 +26,9 @@ final class DenialOfServiceVectorsTest extends SendTestCase
/**
* Test this denial of service attack.
*
* @see http://www.cybsec.com/vuln/PHPMailer-DOS.pdf
* @link http://www.cybsec.com/vuln/PHPMailer-DOS.pdf
*/
public function testDenialOfServiceAttack()
public function testDenialOfServiceAttack1()
{
$this->Mail->Body = 'This should no longer cause a denial of service.';
$this->buildBody();
@ -39,15 +41,15 @@ final class DenialOfServiceVectorsTest extends SendTestCase
* Tests this denial of service attack.
*
* According to the ticket, this should get stuck in a loop, though I can't make it happen.
* @see https://sourceforge.net/p/phpmailer/bugs/383/
* @link https://sourceforge.net/p/phpmailer/bugs/383/
*
* @doesNotPerformAssertions
*/
public function testDenialOfServiceAttack2()
{
//Encoding name longer than 68 chars
// Encoding name longer than 68 chars.
$this->Mail->Encoding = '1234567890123456789012345678901234567890123456789012345678901234567890';
//Call wrapText with a zero length value
// Call wrapText with a zero length value.
$this->Mail->wrapText(str_repeat('This should no longer cause a denial of service. ', 30), 0);
}
}