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

Login para admin #35

Open
pepellou opened this issue Mar 18, 2020 · 2 comments
Open

Login para admin #35

pepellou opened this issue Mar 18, 2020 · 2 comments

Comments

@pepellou
Copy link
Owner

Aunque confiamos en la buena voluntad de la gente, en algún momento estaría bien ponerle una pantalla de login 😅

(yo voy haciendo copias de la BD por si acaso 😬)

@pepellou
Copy link
Owner Author

pepellou commented Apr 7, 2020

Hagámoslo con la opción de "recordar durante X días" con una cookie para que sea fácil entrar sin tener que poner la password cada vez.

@pepellou
Copy link
Owner Author

pepellou commented Apr 7, 2020

La implementación realmente da un poco igual, pero por esbozarla para quien implemente esta issue, una idea sería:

  • Que el index de admin ahora sea la pantalla de login
  • Que la password vaya encriptada en front (hasheada mejor)
  • Que la password hasheada de admin (una única cuenta de admin puede ser lo más sencillo) esté en la BD
  • Que la acción de login consulte a firebase para determinar si es correcto o no
  • Que si el login es exitoso se pase a la vista principal de admin
  • Que las páginas privadas de admin (lo que ahora es index.html que cambiará a otra ruta, y lo que ahora es queries.html) comprueben (en front) si se ha hecho un login válido (¿por cookie?) y si no redirijan al login

Ojo! Esta propuesta es una implementación en front. No es ideal (es muy hackeable), pero ya es mejor que dejar la parte de admin completamente pública.

Esta propuesta puede que sea modificada en el futuro si aparece un backend (que no es descartable), así que si coges esta issue por favor consulta a alguien del equipo, no sea que esta propuesta haya quedado obsoleta 😬

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

No branches or pull requests

1 participant