type hidden wird anscheinend nicht richtig übergeben

[Spunk95]

Danke schonmal für die ganze Hilfe die ich schon hier im Forum bekommen habe
hier mein Problem:

Ich habe ein Seite auf der Kommentare angezeigt werden. Um diese zu löschen werden die erforderlichen Daten mit diesem Formular an eine andere Seite übergeben:

<form action="index.php?s=changecreation&type=comment&action=delete"
 method="post" name="form_comment_delete_<?echo $comment_id;?>">
    <input type="hidden" name="type_id" value="<?echo $comment_id;?>">
</form>

Die Id des Kommentars übergebe ich mit type hidden über POST, damit man sie nicht in der URL ändern kann. Ich weiß allerdings nicht ob es so sicher ist.

Wenn ich dann versuche die versteckte Übergabe auszugeben mit

echo $_POST["type_id"]; 

kommt:

Notice: Undefined index: type_id in /users/mycreationhost/www/change-creation.php on line 3 

Ich weiß aber nicht warum die variable nicht belegt ist...
Was am seltsamsten ist, dass das Script, dass mit dieser Variable arbeitet trotzdem richtig ausgeführt wird und dann eine selbstgeschriebene Fehlermeldung ausgibt, die ebenfalls besagt, dass nicht alle nötigen Angaben vorhanden sind...

Wäre sehr dankbar für Hilfe

[wahsaga]

Die Id des Kommentars übergebe ich mit type hidden über POST, damit man sie nicht in der URL ändern kann. Ich weiß allerdings nicht ob es so sicher ist.

Kein bisschen. Auch POST-Werte sind beliebig manipulierbar.
Du solltest prüfen, ob der jenige, der das Script aufruft, auch dazu berechtigt ist; oder es bspw. mit einem Passwortschutz versehen, so dass es nicht jeder aufrufen kann.

Zum Problem: Wie schickst du das Formular denn überhaupt ab? Ich sehe da keinerlei Submit-Button, und auch kein Eingabefeld, das den Fokus bekommen könnte, so dass ein Abschicken per Return möglich wäre.

[AmicaNoctis]

Hallo,

das Formular hat offenbar außer dem Hidden-Feld keine weiteren Elemente, insbesondere keinen Submit-Button. Ich vermute, dass die Seite aber mindestens ein weiteres Formular enthält und nur dieses abgeschickt wird, wobei auch nur dessen Formularfelder übergeben werden. Bau mal in das Formular mit dem Hidden-Feld einfach einen Submit-Button ein und drück dann dort drauf, dann wirst du sehen, dass es funktioniert.

Gruß,

Amica

[unset]

Schau zunächst einmal, was genau für ein HTML-Quelltext generiert wird. Scheint dieser in Ordnung zu sein, würde ich auch noch einmal schauen, was alles im POST-Array zu sehen ist. Zum Beispiel so:

print_r($_POST); 

[Spunk95]

Danke für die vielen, schnellen Antworten

Kein bisschen. Auch POST-Werte sind beliebig manipulierbar.
Du solltest prüfen, ob der jenige, der das Script aufruft, auch dazu berechtigt ist; oder es bspw. mit einem Passwortschutz versehen, so dass es nicht jeder aufrufen kann.

Ich prüfe vorm auswerten ob die SESSION variable usr_id gesetzt ist, reicht das als Schutz? Und inwiefern kann man POST Werte manipulieren?

Zum Problem: Wie schickst du das Formular denn überhaupt ab? Ich sehe da keinerlei Submit-Button, und auch kein Eingabefeld, das den Fokus bekommen könnte, so dass ein Abschicken per Return möglich wäre.

[/QUOTE]

Abgeschickt wird das Formular mit javascript, da das Formular in einem jquery Dialog steht und es beim Klick auf den Button des Dialogs abgesendet werden soll. Hat auch in anderen Scripten von mir gut geklappt.

Dieser Javascript wird in der gleichen Schleife erstellt wie auch das Formular und sich eigentlich genau auf das oben genannte Formular bezieht:

buttons: {
                                                        "Na sicher": function() { 
                                                            document.form_comment_delete_<?echo $comment_id;?>.submit();
                                                        }, 
                                                        "Nein doch nicht": function() { 
                                                            $(this).dialog("close"); 
                                                        } 
                                                    }

@AmicaNoctis: Habe ich grade ausprobiert und es hat dann auch geklappt. Irgendwie klappt jetzt alles wieder, auch wenn ich den submit button wieder entferne.
Eigentlich ist also alles geklärt, auch wenn ich nicht verstehe, warum es auf einmal funktioniert.

@unset : Danke dir

Also Zusammenfassend: Problem gelöst, aber wie sieht das mit der Sicherheit aus? Wenn man POSt so leicht manipulieren kann, sollte ich vielleicht etwas anderes nutzen?
// Jetzt klappt es auf einmal wieder nicht ohne Submit Button... Das Script wird zwar ausgeführt, aber er gibt eine Fehlermeldung aus...

Danke nochmal an alle

[AmicaNoctis]

Und inwiefern kann man POST Werte manipulieren?

a) Firebug
b) HTML-Code speichern, ändern, im Browser öffnen, abschicken.

[Spunk95]

a) Firebug
b) HTML-Code speichern, ändern, im Browser öffnen, abschicken.

Okay, das Script funktioniert jetzt übrigens perfekt.
Aber wäre ja blöd, wenn man das jetzt so aushebeln kann, was sind denn die alternativen?

Danke schonmal

[wahsaga]

Es ist immer zu prüfen, ob derjenige, der eine bestimmte Aktion ausführen will, dazu auch die nötigen Rechte hat - dazu gibt es keine Alternativen.

[Spunk95]

Es ist immer zu prüfen, ob derjenige, der eine bestimmte Aktion ausführen will, dazu auch die nötigen Rechte hat - dazu gibt es keine Alternativen.

Okay, und wie prüft man das?

[wahsaga]

Bevor du es prüfen kannst, musst du es erst mal definieren.
Wer soll berechtigt sein, Einträge zu löschen, wer nicht?

(Oder halt wie schon erwähnt, Script nur passwortgeschützt aufrufbar machen - und dann einfach sagen, wer das Passwort kennt, hat auch die Berechtigung.)

[Spunk95]

Bevor du es prüfen kannst, musst du es erst mal definieren.
Wer soll berechtigt sein, Einträge zu löschen, wer nicht?

(Oder halt wie schon erwähnt, Script nur passwortgeschützt aufrufbar machen - und dann einfach sagen, wer das Passwort kennt, hat auch die Berechtigung.)

Okay, dann prüfe ich im Script nochmal, ob einem das Kommentar wirklich gehört. Danke für eure Hilfe