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

Para Colaboradores #3

Open
ReinaldoDiasAbreu opened this issue Mar 5, 2019 · 2 comments
Open

Para Colaboradores #3

ReinaldoDiasAbreu opened this issue Mar 5, 2019 · 2 comments

Comments

@ReinaldoDiasAbreu
Copy link
Owner

Olá, tudo bem?

Se você entrou neste repositório, fez o clone do projeto e testou ele. Gostaria muito que você desse alguma opinião desse projeto nesta issue. Também estou disponível para conversar sobre este projeto ou tecnologia em geral.

Muito obrigado!

@bersuc
Copy link
Collaborator

bersuc commented Mar 8, 2019

Oi Reinaldo, tudo bem?
Eu estava olhando aqui e vi que no arquivo php/administrador/banco-admin.php (linha 15) você faz uma consulta usando o LIKE para validar o usuário e senha. Seria também usar um select email, senha em vez de *, embora o sistema seja simples, o ideal é buscar somente as informações que você precisa (nada a mais e nada a menos).
Por motivos de segurança e se tratar de login, seria legal alterar para um
$sql = "select * from admin WHERE email LIKE '".$email."' and senha LIKE '".$senha."'";
Alterando para:
$sql = "select email, senha from admin WHERE email = '".$email."' and senha = '".$senha."'";

O uso do LIKE é somente para quando se conhece parte do conteúdo, e nesse seu caso você já sabe que tem que ser idêntica.

Use WHERE LIKE when only a fragment of a text value is known.

Fonte:
DoFactory

Espero ter colaborado com meus 0,20 centavos, e vou tentar ajudar mais, assim que tiver um tempinho :)
Abs

@ReinaldoDiasAbreu
Copy link
Owner Author

ReinaldoDiasAbreu commented Mar 8, 2019

Muito Obrigado. Assim que possível vou refatorar o código.

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

2 participants