"UPDATE table_name SET w = $1, x = $2, z = $4 WHERE y = $3 RETURNING *",

does not do the same as

"UPDATE table_name SET w = $1, x = $2, y = $3, z = $4 RETURNING *",

It’s 2 am and my mind blanked out the WHERE, and just wanted the numbers neatly in order of 1234.

idiot.

FML.

  • @[email protected]
    link
    fedilink
    21 year ago

    Have a look at an ORM, if you are indeed executing plain SQL like I’m assuming from your comment. Sequelize might be nice to start with. What it does is create a layer between your application and your database. Using which, you can define the way a database object looks (like a class) and execute functions on that. For instance, if you’re creating a library, you could do book.update(), library.addBook(), etc. Since it adds a layer in between, it also helps you prevent common vulnerabilities such as SQL injection. This is because you aren’t writing the SQL queries in the first place. If you want to know more, let me know.

    • @dreklyOP
      link
      English
      21 year ago

      Thanks, I’ll look into it! I’m interested in why you got downvoted though! 😅

      • @[email protected]
        link
        fedilink
        31 year ago

        I didn’t downvote but some people like ideologically dislike orms. The reasons I’ve heard are usually “I can write better SQL by hand”, “I don’t want to use/learn another library”, “it has some limitations”

        Those things can be true. Writing better SQL by hand definitely is a big “it depends”, though.

        • @[email protected]
          link
          fedilink
          21 year ago

          I can see why people might dislike them. Adds some bloat perhaps. But at the same time, I like the idea that my input is definitely sanitised since the ORM was written by people who know what they’re doing. That’s not to say it won’t have any vulnerabilities at all, but the chance of them existing is a lot lower than when I write the queries by hand. A lapse of judgement is all it takes. Even more relevant for beginning developers who might not be aware of such vulnerabilities existing.

          • @dreklyOP
            link
            English
            11 year ago

            For a personal tool that runs locally I can handle some bloat in the name of safety!

            • @[email protected]
              link
              fedilink
              11 year ago

              Mostly just safety from yourself/your own little errors in input, but it can’t hurt for sure! Input sanitation is mostly relevant to fend off script kiddies. Relevant xkcd

      • @[email protected]
        link
        fedilink
        2
        edit-2
        1 year ago

        Short story, haters gonna hate ¯\_(ツ)_/¯ Long story, see my comment to the commenter below you. :)