Ir para conteúdo

POWERED BY:

Arquivado

Este tópico foi arquivado e está fechado para novas respostas.

laerciocalheiros

Sistema de login bugado

Recommended Posts

Criei um site com um sistema de login que eu costumo usar em quase todos os sites que eu faço, porem no ultimo que eu fiz está ocorrendo um erro onde alguns usuários ao fazer login estão indo para outras contas e até as vezes de acordo com que a pessoa vai atualizando o site a conta vai trocando. Já  refiz o código mas o problema continua e então estou aqui para ver se alguém  sabe o que pode ser.

 

Segue o código abaixo: 

// Includes
include '../php/config.php';
include '../php/functions.php';

// Verificando se o usuário já está logado
if(isLoggedIn()){ die(json_encode(array('error' => 'Você já está logado.'))); }

// Data request
$username = $_POST['username'];
$password = md5($_POST['password'].base64_encode($username));

// Validação dos dados
if($_POST){
	$sql = $pdo->prepare("SELECT * FROM usuarios WHERE nick= ? AND senha= ? LIMIT 1");
	$sql->bindValue(1, $username);
	$sql->bindValue(2, $password);
	$sql->execute();

	if($sql->rowCount() < 1){
		$error['error'] = 'Usuário ou senha incorreto.';
		die(json_encode($error));
	}

	// Verificando se o usuário está banido.
	$row = $sql->fetch();

	if($row['banido'] != 'n'){
		if($row['ban_termino'] < time()){
			$error['error'] = "Você foi banido até o dia ".date('d/m/y - h:i:s', $row['ban_termino']). "\r Motivo: ".$row['ban_motivo'];
			die(json_encode($error));
		}else{
			// Removendo o banimento do usuário.
			$upd = $pdo->prepare("UPDATE usuarios SET banido='n', ban_motivo='', ban_termino='' WHERE nick= ?");
			$upd->bindValue(1, $row['nick']);
			$upd->execute();

			// Atualizando ip e data 
			$upd1 = $pdo->prepare("UPDATE usuarios SET acesso_data= ?, acesso_ip= ?, acesso_ua= ?, acesso_gl= ? WHERE nick= ?");
			$upd1->bindValue(1, time());
			$upd1->bindValue(2, $_SERVER['REMOTE_ADDR']);
			$upd1->bindValue(3, $_SERVER['HTTP_USER_AGENT']);
			$upd1->bindValue(4, $_SERVER['HTTP_ACCEPT_LANGUAGE']);
			$upd1->bindValue(5, $row['nick']);
			$upd1->execute();

			$_SESSION['user.nick'] = $row['nick'];
			$_SESSION['user.id'] = $row['id'];

			if($upd1){
				$error['success'] = 'Usuário logado com sucesso!';
				echo json_encode($error);
			}else{
				$error['error'] = 'Houve algum erro no sistema. Tenta novamente mais tarde.';
				echo json_encode($error);
			}
		}
	}else{
		// Atualizando ip e data 
		$upd1 = $pdo->prepare("UPDATE usuarios SET acesso_data= ?, acesso_ip= ?, acesso_ua= ?, acesso_gl= ? WHERE nick= ?");
		$upd1->bindValue(1, time());
		$upd1->bindValue(2, $_SERVER['REMOTE_ADDR']);
		$upd1->bindValue(3, $_SERVER['HTTP_USER_AGENT']);
		$upd1->bindValue(4, $_SERVER['HTTP_ACCEPT_LANGUAGE']);
		$upd1->bindValue(5, $row['nick']);
		$upd1->execute();

		$_SESSION['user.nick'] = $row['nick'];
		$_SESSION['user.id'] = $row['id'];

		if($upd1){
			$error['success'] = 'Usuário logado com sucesso!';
			echo json_encode($error);
		}else{
			$error['error'] = 'Houve algum erro no sistema. Tenta novamente mais tarde.';
			echo json_encode($error);
		}
	}
}

Função onde eu verifico se o usuário está logado:
 

function isLoggedIn(){
	session_start();

	if(!isset($_SESSION['user.nick']) && $_SESSION['user.nick'] == ''){
		return false
	}

	return true;
}

 

Compartilhar este post


Link para o post
Compartilhar em outros sites

só uma pergunta pra que butar isso separado? se passar o if(isset...) na própria página de login vai ocupar o mesmo tanto de linha lá e evita função extra. sei lá mas será que não é mió hehehe

Compartilhar este post


Link para o post
Compartilhar em outros sites
41 minutos atrás, marsolim disse:

só uma pergunta pra que butar isso separado? se passar o if(isset...) na própria página de login vai ocupar o mesmo tanto de linha lá e evita função extra. sei lá mas será que não é mió hehehe

Eu utilizo ele em várias páginas, então é melhor ele ser uma função

Compartilhar este post


Link para o post
Compartilhar em outros sites

não tá faltando um else aí não? desse tipo que tá vai sempre retornar true porque ele vai passar pela condição if e continuar até chegar no true. talvez assim

function isLoggedIn(){
        session_start();

        if(!isset($_SESSION['user.nick']) && $_SESSION['user.nick'] == ''){
            return false
    }else{
            return true;
        }
}

ou então coloca um break; depois de return false pra interromper o código.

Compartilhar este post


Link para o post
Compartilhar em outros sites
14 horas atrás, marsolim disse:

não tá faltando um else aí não? desse tipo que tá vai sempre retornar true porque ele vai passar pela condição if e continuar até chegar no true. talvez assim


function isLoggedIn(){
        session_start();

        if(!isset($_SESSION['user.nick']) && $_SESSION['user.nick'] == ''){
            return false
    }else{
            return true;
        }
}

ou então coloca um break; depois de return false pra interromper o código.


O código da função que ele colocou está correto, são boas praticas de programação e se chama Object Calisthenics.

Entenda sobre:

 

https://imasters.com.br/linguagens/php/como-melhorar-seus-codigos-usando-object-calisthenics/?trace=1519021197&source=single
 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Olhando rápido, só achei estranho esse LIMIT 1

SELECT * FROM usuarios WHERE nick= ? AND senha= ? LIMIT 1

E considerando que você disse que o usuário vai trocando quando atualiza a pagina, não teria duplicação de usuário e senha no banco de dados?

Compartilhar este post


Link para o post
Compartilhar em outros sites
1 hora atrás, EdCesar disse:

Olhando rápido, só achei estranho esse LIMIT 1


SELECT * FROM usuarios WHERE nick= ? AND senha= ? LIMIT 1

E considerando que você disse que o usuário vai trocando quando atualiza a pagina, não teria duplicação de usuário e senha no banco de dados?

 

Já verifiquei isso também, e está tudo correto, estou achando que são essas sessions 

Compartilhar este post


Link para o post
Compartilhar em outros sites

Por um acaso você usa  

$_SESSION['user.nick'] e $_SESSION['user.id']

em algum outro sistema, que esteja no mesmo servidor? Se sim, esse é o problema, um sistema estaria sobrescrevendo o valor de outro sistema. Ai a solução seria colocar um prefixo,  por exemplo:

$_SESSION['xpto.user.nick] = $row['nick'];

 

Compartilhar este post


Link para o post
Compartilhar em outros sites
2 horas atrás, EdCesar disse:

Por um acaso você usa  


$_SESSION['user.nick'] e $_SESSION['user.id']

em algum outro sistema, que esteja no mesmo servidor? Se sim, esse é o problema, um sistema estaria sobrescrevendo o valor de outro sistema. Ai a solução seria colocar um prefixo,  por exemplo:


$_SESSION['xpto.user.nick] = $row['nick'];

 

 

Tem não, eu já modifiquei pensando que era isso também.

Compartilhar este post


Link para o post
Compartilhar em outros sites

  • Conteúdo Similar

    • Por Rafael_Ferreira
      Não consigo carregar a imagem do captcha do meu formulário. Foi testado com o xampp e easyphp. Também não carregou a imagem de outros captcha. 
       
       
    • Por luiz monteiro
      Olá, tudo bem?
       
      Estou melhorando meu conhecimento em php e mysql e, me deparei com o seguinte. A tabela da base de dados tem um campo do tipo varchar(8) o qual armazena números. Eu não posso alterar o tipo desse campo. O que preciso é fazer um select para retornar o números que contenham zeros a direita ou a esquerda.
      O que tentei até agora
       
      Ex1
      $busca = $conexao->prepare("select campo form tabela where (campo = :campo) ");
      $busca->bindParam('campo', $_REQUEST['campo_form']);
       
      Se a direita da string $_REQUEST['campo_form'] termina ou inicia com zero ou zeros, a busca retorna vazio.
      Inseri dados numéricos, da seguinte maneira para testar: 01234567;  12345670: 12345678: 12340000... entre outros nessa coluna. Todos os valores que não terminam ou não iniciam com zero ou zeros, o select funciona.
       
       
      Ex2
      $busca = $conexao->prepare("select campo form tabela where (campo = 0340000) ");
      Esse número está cadastrado, mas não retorna.
       
      Ex3
      $busca = $conexao->prepare("select campo form tabela where (campo = '02340001' ) ");
      Esse número está cadastrado, mas não retorna.
       
       
      Ex4
      $busca = $conexao->prepare("select campo form tabela where (campo like 2340000) ");
      Esse número está cadastrado, mas não retorna.
       
      Ex5
      $busca = $conexao->prepare("select campo form tabela where (campo like '12340000') ");
      Esse número está cadastrado, mas não retorna.
       
      Ex6
      $busca = $conexao->prepare("select campo form tabela where (campo like '"12340000"' ) ");
      Esse número está cadastrado, mas não retorna.
       
       
      Ex7
      $busca = $conexao->prepare("select campo form tabela where (campo like :campo) ");
      $busca->bindParam('campo', $_REQUEST['campo_form'])
      Não retorna dados.
       
      O  $_REQUEST['campo_form'] é envio via AJAX de um formulário. 
      Usei o gettype para verificar o post, e ele retorna string.
      Fiz uma busca com número 12345678 para verificar o que o select retorna, e também retrona como string.
       
      Esse tipo de varchar foi usado porque os números que serão gravados nesse campo,  terão zeros a direita ou na esquerda. Os tipos number do mysql não gravam zeros, então estou usando esse. O problema é a busca.
      Agradeço desde já.
       
       
    • Por daemon
      Boa tarde,
       
      Eu tenho uma rotina que faz uma leitura do arquivo .xml de vários sites.

      Eu consigo pegar o tópico e a descrição, e mostrar a imagem que esta na pagina do link.
      Para isso utilizo esta função:
      function getPreviewImage($url) { // Obter o conteúdo da página $html = file_get_contents($url); // Criar um novo objeto DOMDocument $doc = new DOMDocument(); @$doc->loadHTML($html); // Procurar pela tag meta og:image $tags = $doc->getElementsByTagName('meta'); foreach ($tags as $tag) { if ($tag->getAttribute('property') == 'og:image') { return $tag->getAttribute('content'); } } // Se não encontrar og:image, procurar pela primeira imagem na página $tags = $doc->getElementsByTagName('img'); if ($tags->length > 0) { return $tags->item(0)->getAttribute('src'); } // Se não encontrar nenhuma imagem, retornar null return null; } // Uso: $url = "https://example.com/article"; $imageUrl = getPreviewImage($url); if ($imageUrl) { echo "<img src='$imageUrl' alt='Preview'>"; } else { echo "Nenhuma imagem encontrada"; }  
      Mas estou com um problema, esta funcão funciona quando coloco em uma pagina de teste.php. Preciso mostrar em uma página inicial diversas fotos de todos os links. (No caso acima só funciona 1).
    • Por violin101
      Caros amigos, saudações.
       
      Por favor, me permita tirar uma dúvida com os amigos.

      Tenho um Formulário onde o Usuário digita todos os Dados necessários.

      Minha dúvida:
      --> como faço após o usuário digitar os dados e salvar, o Sistema chamar uma Modal ou mensagem perguntando se deseja imprimir agora ?

      Grato,
       
      Cesar
    • Por Carcleo
      Tenho uma abela de usuarios e uma tabela de administradores e clientes.
      Gostaria de uma ajuda para implementar um cadastro
       
      users -> name, login, passord (pronta) admins -> user_id, registratiom, etc.. client -> user_id, registratiom, etc...
      Queria ajuda para extender de user as classes Admin e Client
      Olhem como estáAdmin
      <?php namespace App\Models; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Database\Eloquent\Model; class Admin extends User {     use HasFactory;            protected $fillable = [         'name',         'email',         'password',         'registration'     ];      private string $registration;     public function create(         string $name,          string $email,          string $password,         string $registration     )     {         //parent::create(['name'=>$name, 'email'=>$email, 'password'=>$password]);         parent::$name = $name;         parent::$email = $email;         parent::$password = $password;         $this->registration = $registration;     } } User
      <?php namespace App\Models; // use Illuminate\Contracts\Auth\MustVerifyEmail; use Illuminate\Database\Eloquent\Factories\HasFactory; use Illuminate\Foundation\Auth\User as Authenticatable; use Illuminate\Notifications\Notifiable; use Illuminate\Database\Eloquent\Relations\BelongsToMany; class User extends Authenticatable {     /** @use HasFactory<\Database\Factories\UserFactory> */     use HasFactory, Notifiable;     static string $name;     static string $email;     static string $password;     /**      * The attributes that are mass assignable.      *      * @var list<string>      */     protected $fillable = [         'name',         'email',         'password',     ];          /**      * The attributes that should be hidden for serialization.      *      * @var list<string>      */     protected $hidden = [         'remember_token',     ];     /**      * Get the attributes that should be cast.      *      * @return array<string, string>      */     protected function casts(): array     {         return [             'email_verified_at' => 'datetime',             'password' => 'hashed',         ];     }          public function roles() : BelongsToMany {         return $this->belongsToMany(Role::class);     }       public function hasHole(Array $roleName): bool     {                 foreach ($this->roles as $role) {             if ($role->name === $roleName) {                 return true;             }         }         return false;     }         public function hasHoles(Array $rolesName): bool     {                 foreach ($this->roles as $role) {             foreach ($rolesName as $rolee) {             if ($role->name === $rolee) {                 return true;             }          }         }         return false;     }         public function hasAbility(string $ability): bool     {         foreach ($this->roles as $role) {             if ($role->abilities->contains('name', $ability)) {                 return true;             }         }         return false;     }     } Como gravar um Admin na tabela admins sendo que ele é um User por extensão?
      Tentei assim mas é claro que está errado...
      public function store(Request $request, Admin $adminModel) {         $dados = $request->validate([             "name" => "required",             "email" => "required|email",             "password" => "required",             "registration" => "required"         ]);         $dados["password"] =  Hash::make($dados["password"]);                  $admin = Admin::where("registration",  $dados["registration"])->first();                  if ($admin)              return                    redirect()->route("admin.new")                             ->withErrors([                                 'fail' => 'Administrador já cadastrados<br>, favor verificar!'                   ]);                            $newAdmin = $adminModel->create(                                    $dados['name'],                                    $dados['email'],                                    $dados['password'],                                    $dados['registration']                                 );         dd($newAdmin);         $adminModel->save();         //$adminModel::create($admin);                  return redirect()->route("admin.new")->with("success",'Cadastrado com sucesso');     }  
×

Informação importante

Ao usar o fórum, você concorda com nossos Termos e condições.