Egulias\EmailValidator\Helper\SmtpSocketHelper

Complexity

Total Complexity

11

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	isResource()	0	4	1
A	open()	0	4	1
A	write()	0	8	2
A	getResponseCode()	0	15	4
A	close()	0	10	2

<?php

namespace Egulias\EmailValidator\Helper;

class SmtpSocketHelper
{
    /**
     * @var int
     */
    private $port;

    /**
     * @var int
     */
    private $timeout;

    /**
     * @var resource
     */
    private $handle;

    public function __construct($port = 25, $timeout = 15)
    {
        $this->port = $port;
        $this->timeout = $timeout;
    }

    /**
     * Checks is resource
     *
     * @return bool
     */
    public function isResource()
    {
        return is_resource($this->handle);
    }

    /**
     * Opens resource
     *
     * @param string $hostname
     * @param int $errno
     * @param string $errstr
     */
    public function open($hostname, &$errno, &$errstr)
    {
        $this->handle = @fsockopen($hostname, $this->port, $errno, $errstr, $this->timeout);
    }

    /**
     * Writes message
     *
     * @param string $message
     *
     * @return bool|int
     */
    public function write($message)
    {
        if (!$this->isResource()) {
            return false;
        }

        return @fwrite($this->handle, $message);
    }

    /**
     * Get last response code
     *
     * @return int
     */
    public function getResponseCode()
    {
        if (!$this->isResource()) {
            return -1;
        }

        $data = '';
        while (substr($data, 3, 1) !== ' ') {
            if (!($data = @fgets($this->handle, 256))) {
                return -1;
            }
        }

        return intval(substr($data, 0, 3));
    }

    /**
     * Closes resource
     */
    public function close()
    {
        if (!$this->isResource()) {
            return;
        }

        @fclose($this->handle);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

        $this->handle = null;
    }
}