613
Works if manually restarted by an intern from time to time (lemmy.world)
submitted 1 year ago by to c/programmer_humor@programming.dev
51 comments fedilink hide all child comments
 
you are viewing a single comment's thread
view the rest of the comments
[–] 18 points 1 year ago (5 children)

Yes, had the same happen. Something that should be simple failing for stupid reasons.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 5 child comments
    •
  • [–] 13 points 1 year ago (4 children)

    Well it's not that simple... Because whoever wrote that made it way too complicated (and the production version has been tweaked without updating the dev too)

    A clean rewrite with some guard clauses helped remove the haduken ifs and actually zipping the file outside of the zipped directory helped a lot

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 4 child comments
    •
  • [–] 7 points 1 year ago (1 child)

    I mean, I have to say I've hastened my own demise (in program terms) by over-engineering something that should be simple. Sometimes adding protective guardrails actually causes errors when something changes.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 1 child comment
    •
  • [–] 3 points 1 year ago (1 child)

    Am I understanding that last part correctly?

    [...] and actually zipping the file outside of the zipped directory helped a lot

    Did they just automatically create a backup zip-bomb in their script‽

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 1 child comment
    •
  • [–] 9 points 1 year ago

    I oversimplified it but the actual process was to zip files to send to an FTP server

    The cron zipped the files to send in the same directory as the zipped files, then sent the zip, then deleted the zip

    Looks fine, right? But what if the FTP server is slow and uploading take more time than the hourly cron dispatch? You now have a second script that zip all the folder, with the previous zip file, which will slow down the upload, etc...

    I believe may have been started by an FTP upload erroring out and forcing an early return without having a cleanup, and progressively got worse

    ... I suppose this happened. The logs were actually broken and didn't actually add the message part of the error object, and only logging the memory address to it

  • permalink
  • fedilink
  • source
  • parent