Merge pull request #183 from robmachado/master

[UPD] ISSUE #182 Melhoria na manipulação de chaves temporárias
nfephp-org · Jun 7, 2018 · 8988051 · gersonfs · Jun 7, 2018 · 8988051
2 parents 767d4f8 + 430d780
commit 8988051
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 39 deletions.
diff --git a/src/DOMImproved.php b/src/DOMImproved.php
@@ -155,7 +155,7 @@ public function getChave($nodeName = 'infNFe')
      * Serão inclusos erros na array $erros[] sempre que a tag for obrigatória e
      * nenhum parâmetro for passado na variável $content e $force for false
      * @param \DOMElement $parent
-     * @param string $name
+     * @param string|null $name
      * @param string|float|null $content
      * @param boolean $obrigatorio
      * @param string $descricao
@@ -170,7 +170,7 @@ public function addChild(
         $descricao = '',
         $force = false
     ) {
-        if ($content === null) {
+        if ($content === null || empty($name)) {
             return;
         }
         $content = (string) $content;

diff --git a/src/Soap/SoapBase.php b/src/Soap/SoapBase.php
@@ -221,12 +221,6 @@ public function loadCA($capath)
     public function setEncriptPrivateKey($encript = true)
     {
         $this->encriptPrivateKey = $encript;
-        if (null !== $this->filesystem) {
-            $this->removeTemporarilyFiles();
-        }
-        if (null !== $this->certificate) {
-            $this->saveTemporarilyKeyFiles();
-        }
         return $this->encriptPrivateKey;
     }
 
@@ -451,6 +445,7 @@ private function getStringAttributes($namespaces = [])
      */
     public function saveTemporarilyKeyFiles()
     {
+        //certs already exists
         if (!empty($this->certsdir)) {
             return;
         }
@@ -462,16 +457,17 @@ public function saveTemporarilyKeyFiles()
         if (empty($this->filesystem)) {
             $this->setTemporaryFolder();
         }
+        //clear dir cert
+        $this->removeTemporarilyFiles();
         $this->certsdir = $this->certificate->getCnpj() . '/certs/';
         $this->prifile = $this->certsdir . Strings::randomString(10) . '.pem';
         $this->pubfile = $this->certsdir . Strings::randomString(10) . '.pem';
         $this->certfile = $this->certsdir . Strings::randomString(10) . '.pem';
         $ret = true;
+        //load private key pem
         $private = $this->certificate->privateKey;
         if ($this->encriptPrivateKey) {
-            //cria uma senha temporária ALEATÓRIA para salvar a chave primaria
-            //portanto mesmo que localizada e identificada não estará acessível
-            //pois sua senha não existe além do tempo de execução desta classe
+            //replace private key pem with password
             $this->temppass = Strings::randomString(16);
             //encripta a chave privada entes da gravação do filesystem
             openssl_pkey_export(
@@ -509,35 +505,8 @@ public function removeTemporarilyFiles()
             return;
         }
         $contents = $this->filesystem->listContents($this->certsdir, true);
-        //define um limite de $waitingTime min, ou seja qualquer arquivo criado a mais
-        //de $waitingTime min será removido
-        //NOTA: quando ocorre algum erro interno na execução do script, alguns
-        //arquivos temporários podem permanecer
-        //NOTA: O tempo default é de 45 minutos e pode ser alterado diretamente nas
-        //propriedades da classe, esse tempo entre 5 a 45 min é recomendável pois
-        //podem haver processos concorrentes para um mesmo usuário. Esses processos
-        //como o DFe podem ser mais longos, dependendo a forma que o aplicativo
-        //utilize a API. Outra solução para remover arquivos "perdidos" pode ser
-        //encontrada oportunamente.
-        $dt = new \DateTime();
-        $tint = new \DateInterval("PT" . $this->waitingTime . "M");
-        $tint->invert = 1;
-        $tsLimit = $dt->add($tint)->getTimestamp();
         foreach ($contents as $item) {
-            if ($item['type'] == 'file') {
-                if ($item['path'] == $this->prifile
-                    || $item['path'] == $this->pubfile
-                    || $item['path'] == $this->certfile
-                ) {
-                    $this->filesystem->delete($item['path']);
-                    continue;
-                }
-                $timestamp = $this->filesystem->getTimestamp($item['path']);
-                if ($timestamp < $tsLimit) {
-                    //remove arquivos criados a mais de 45 min
-                    $this->filesystem->delete($item['path']);
-                }
-            }
+            $this->filesystem->delete($item['path']);
         }
     }
 

diff --git a/src/Soap/SoapCurl.php b/src/Soap/SoapCurl.php
@@ -56,6 +56,8 @@ public function send(
         $request = '',
         $soapheader = null
     ) {
+        //check or create key files
+        //before send request
         $this->saveTemporarilyKeyFiles();
         $response = '';
         $envelope = $this->makeEnvelopeSoap(