Jump to content

Recommended Posts

Hi Guys,

 

Well I started of with my login page using prepared statements, but at this point in time it does not do anything! Does not tell me weither the username and password is incorrect or right, does not move on to the next page, just nothing. Anything I am missing here?

login.php

Link to comment
https://forums.phpfreaks.com/topic/275684-coding-not-moving-forward/
Share on other sites

 mysql_real_escape_string($username = $_POST["username"]);

 

This is a little funky to me, it should look like the following...

$username = mysql_real_escape_string($_POST["username"]);

 

Not only that, you don't need to use MRES, you are using query parameters kind of protects you against SQL Injection for you.

 

Chances are, your query could be failing, you don't really check if it fails. The while statement is relatively pointless too.

 

You are echoing data before your doctype declaration, this is invalid HTML however I believe it'll still output visible data to the browser.

 

On a side note, your password is stored in plain text which is a huge security issue. You should use a hashing algorithm such as hash. You also gave your database connection details, you shouldn't do that especially if it's publicly accessible.

 

I've given you a few things to sort out, perhaps this will give you a step in the right direction. Next time, you don't need to attach a file, you can insert code directly onto the forum. In the editor there is a little icon that looks a bit like code.png using that will open a window for you to paste your code and the syntax etc :)

Well I made som changes to my coding, seems I have a issue with my javascript function at the onclick event. Basically I am added <a href="javascript:login();">test</a></td> to test, that works as I am getting my message "submitting", and also there seems to be some issue with mysqli connection as I get my message " failed connection ".

 

<!DOCTYPE html>
<html>
    <script language = "javascript">
        function login()
        {
            alert("submitting");
            document.login.submit();
        }
    </script>
<form name="login" action="login.php?action=login" method="post">
    <table width="300" border="0" align="center" cellpadding="0" cellspacing="1" bgcolor="#CCCCCC">
        <tr>
            <td>
            <table width="100%" border="0" cellpadding="3" cellspacing="1" bgcolor="#FFFFFF">
            <tr>
            <td colspan="3"><strong>Radius Login </strong></td>
        </tr>
        <tr>
            <td width="78">Username</td>
            <td width="6">:</td>
            <td width="294"><input name="username" type="text" id="username"></td>
        </tr>
        <tr>
            <td>Password</td>
            <td>:</td>
            <td><input name="password" type="password" id="password"></td>
        </tr>
        <tr>
            <td> </td>
            <td> </td>
           <td><input type="button" name="Submit" value="Login" onclick="javascript:login();"/><a href="javascript:login();">test</a></td>
        </tr>
    </table>
        </td>
        </tr>
    </table>
</form>
</html>

<?php
require_once 'config.php';

error_reporting(E_ALL);


//Connection to my database
$mysqli = new mysqli(DB_HOST,DB_USER,DB_PASSWORD);

if(mysqli_connect_errno())
            {
                printf("Connection failed: %s\n", mysqli_connect_errno());
                exit();
            }


if($_GET['action'] == 'login')
    {
            
            //prepared statement
            if($stmt = $mysqli->prepare("select * from login where username = ? and password = ?"))
            {
                
                $username = $_POST["username"];
                $password = $_POST["password"];
                
                //bind my parameters
                $stmt->bind_param("ss",$username,$password);
                
                //execute query
                $stmt->execute();
                
                //bind the result variables
                $stmt->bind_result($username,$password);
                
                //Store my values
                $stmt->store_result();
                
                //fetch values
                if($stmt->fetch())
                       {
                            $_SESSION["username"] = $username;
                            header("location : index.php");
                            exit();
                       }
                
                
                
                else
                {
                    echo "Username or password is incorrect";
                }
                
                $stmt->close();
                $stmt->free_result();
                
            }
            else
            {
                echo "Failed connection";
            }
    }
    else
    {
        echo("failed");
    }
        $mysqli->close();

?>

This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.