Skip to content
Snippets Groups Projects

new feature: delete cron from database

Merged Henoch Einbier requested to merge g2zer0/VMSone:patch-18 into dev-3.0
1 unresolved thread

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Hi, da hast du aber einen groben Denkfehler... Die cron.php ist ohne Login und Autorisierung. Da einfach mit ?del=true&id=2 den cron 2 löschen zu können ist da nicht so das feine ;-)

      Ebenfalls gibt es kein reloads_mail im Standard ...

      Ich würde empfehlen bei Änderungen vorher ein "Cleanen Checkout der aktuellen git Version zu nehmen.

    • Please register or sign in to reply
  • guck mal in zeile 7: if ($_GET['pw'] != $system['cron_pw']) die('Zugriff verweigert!');

    die cron.php ist doch mit dem cron pw abgesichert. ohne das geht gar nix.

    stimmt, reloads_mail war extra ;)

  • korrekt, übersehen ;)

    Kannst du noch bisschen cleanup betreiben ... sodass dateien möglichst unverändert sind an leerzeilen etc

    sowie dann error_reporting(E_ALL & ~E_NOTICE); und das _reloads_mail noch entfernen

    die $zeit = time(); anpassung bringt keinen vorteil ändert aber den code...

    sobald du das angepasst hast kann ich es zusammen mit dem anderemn merge request mergen da beides ja voneinander abhängt

  • Henoch Einbier added 1 commit

    added 1 commit

    Compare with previous version

  • merged

  • Joel Kuder mentioned in commit 419633e0

    mentioned in commit 419633e0

Please register or sign in to reply