Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tarea #1439 - crear clase para encriptar #1548

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

elguitarraverde
Copy link
Contributor

Descripción

  • He usado una clase de laravel que funciona correctamente y la he adaptado a FacturaScripts.
  • He modificado la clase Tools para añadir los metodos encriptar y desencriptar
  • He añadido logica en el index para que se use la variable de entorno desde el archivo .env cuando estamos en desarrollo(FS_DEBUG=true) o para que se use la variable de entorno del hosting cuando estamos en producción(FS_DEBUG=false).
  • No lo he implementado en el config.php porque carecería de seguridad en el caso que se vulnere el servidor y se tenga acceso a este archivo. Forzando a configurar la variable de entorno en el hosting no hay forma de que los atacantes sepan la variable APP_KEY y no podrán desencriptar nada.
  • Ahora es necesaria la extensión "openssl" que se encarga de encriptar y desencriptar.
  • Ahora es necesario el paquete "vlucas/phpdotenv" que se encarga de añadir las variables de entorno desde el archivo .env

¿Cómo has probado los cambios?

Toda modificación debe haber sido mínimamente probada. Marca o describe las pruebas que has realizado:

  • He revisado mi código antes de enviarlo.
  • He probado que funciona correctamente en mi PC.
  • He probado que funciona correctamente con una base de datos vacía.
  • He ejecutado los tests unitarios.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant