HTTP ERROR 500

Door Onbekende gebruiker op 17-12-2017 11:43 gewijzigd op 17-12-2017 11:51

3.736 views


<?php
mysql_connect("localhost","-","-");
mysql_select_db("-");


if(isset($_POST['submit']))
{
	$name = $_FILES['file']['name'];
	$temp = $_FILES['file']['tmp_name'];
	
	move_uploaded_file($temp,"uploaded/".$name);
	$url = "/uploaded/$name";
	mysql_query("INSERT INTO `videos` VALUES ('','$name','$url')");
}

?>
<!doctype html>
<html>
	<head>
                <meta name="viewport" content="width=device-width, initial-scale=1">
		<title>www.test.nl</title>
		<link rel="stylesheet" type="text/css" href="style.css" />
	</head>
	<body>
		<div class="container">
			<div class="header">
				<h1>www.test.nl</h1>
			</div><br/>

			<div class="content">
	<center><h2>Upload Your Video</h2><br/>
	<form action="index.php" method="POST" enctype="multipart/form-data">
	<input type="file" name="file" /><br/><br/>
    <input type="submit" name="submit" value="Upload!" /><br/><br/>
</form></center>

<?php

if(isset($_POST['submit']))
{
$query = mysql_query("SELECT * FROM `videos`");
while($row = mysql_fetch_assoc($query))
{
	$id = $row['id'];
	$name = $row['name'];
}
	echo "Your upload is created <a href='watch.php?id=$id' target='blank_'>CLICK</a>";
}

?>
				</div>
<br/><br/>
<br/>
<br/>

			</div>
			<div id="footer">
				Copyright &copy; 2017.
			</div>
		</div>
	</body>
</html>

Ik krijg de code "HTTP ERROR 500" weet iemand waarom?

Ariën

17-12-2017 12:06 gewijzigd op 17-12-2017 12:08

Bekijk je error_log eens. ;-)

Ik heb een sterk vermoeden dat je hosting de mysql-functies niet meer ondersteund sinds een upgrade naar PHP7. Dan zul je moeten overstappen op de functies van MySQLi of PDO.

Verder is je upload-systeem gigantisch lek, omdat je niks controleert. Iemand kan eenvoudig een desastreus php-script uploaden.

Onbekende gebruiker

17-12-2017 12:10

Niet zo snel aan gedacht ;p

"Error: Call to undefined function mysql_connect()"

Ariën

17-12-2017 12:13

Inderdaad.
"Time to upgrade your functions" ;-)

Onbekende gebruiker

17-12-2017 16:40

Bijgwerkt werkt niet. Ik denk dat ik iets over het hoofd zie.


<?php
$con = mysqli_connect("localhost","","","");

// Check connection
if (mysqli_connect_errno())
  {
  echo "Failed to connect to MySQL: " . mysqli_connect_error();
  }

if(isset($_POST['submit']))
{
	$name = $_FILES['file']['name'];
	$temp = $_FILES['file']['tmp_name'];
	
	move_uploaded_file($temp,"uploaded/".$name);
	$url = "/uploaded/$name";
	mysqli_query("INSERT INTO `videos` VALUES ('','$name','$url')");
}

?>
<!doctype html>
<html>
	<head>
                <meta name="viewport" content="width=device-width, initial-scale=1">
		<title>www.test.nl</title>
		<link rel="stylesheet" type="text/css" href="style.css" />
	</head>
	<body>
		<div id="container">
			<div id="header">
				<h1>www.test.nl</h1>
			</div><br/>

			<div id="content">
	<center><h2>Upload Your Video</h2><br/>
	<form action="index.php" method="POST" enctype="multipart/form-data">
	<input type="file" name="file" /><br/><br/>
    <input type="submit" name="submit" value="Upload!" /><br/><br/>
</form></center>

<?php

if(isset($_POST['submit']))
{
$query = mysqli_query("SELECT * FROM `videos`");
while($row = mysqli_fetch_assoc($query))
{
	$id = $row['id'];
	$name = $row['name'];
}
	echo "Your upload is created <a href='watch.php?id=$id' target='blank_'>CLICK</a>";
}

?>
				</div>
<br/><br/>
<br/>
<br/>

			</div>
			<div id="footer">
				Copyright &copy; 2017.
			</div>
		</div>
	</body>
</html>

Ariën

17-12-2017 17:00

Wat gebeurt er? Wat is de foutmelding?

Onbekende gebruiker

17-12-2017 17:16

Hij maakt geen $id aan staat wel dat hij geupload is. (Your upload is created CLICK)
Hij komt ook niet in mysql te staan.

Ariën

17-12-2017 17:18 gewijzigd op 17-12-2017 17:20

Voeg foutafhandeling aan je queries toe met: mysqli_error($con)

En zeg nooit zonder een controle te hebben uitgevoerd dat een actie gelukt is.

Ivo P

17-12-2017 18:30

mysqli_query heeft 2 argumenten nodig.

op regel 16 zie ik er maar 1.
je mist $con

Verder is je script op meerdere punten gevaarlijk. ook vatbaar voor sql - injectie

Ariën

17-12-2017 18:51 gewijzigd op 17-12-2017 18:54

En iedereen kan elke soort bestand uploaden. Dus ook gevaarlijke PHP-scripts.
Ik hoop van harte dat dit niet live staat voor iedereen.

Verder opmerkelijk dat je twee keer kijkt of je formulier uitgevoerd wordt.
Dit kan je prima doen met een enkele if($_SERVER['REQUEST_METHOD']=="POST") {

Dat is ook een logische stap, want je wilt juist kijken of je formulier verstuurd is, en niet of er op een knop is gedrukt.

Ivo P

17-12-2017 21:50

mysqli_query heeft 2 argumenten nodig.

op regel 16 zie ik er maar 1.
je mist $con

Verder is je script op meerdere punten gevaarlijk. ook vatbaar voor sql - injectie

[size=xsmall]Toevoeging op 17/12/2017 21:52:02:[/size]

mmm

even tussen door: ik krijg
"Reageren op HTTP ERROR 500
Expected response code 250 but got code "452", with message "452 4.3.1 Insufficient system storage "

als reactie bij het posten op phphulp, ipv een "je wordt over 4 seconden doorgstuurd"

Reageren

Inloggen om te reageren