• x00z@lemmy.world
    link
    fedilink
    English
    arrow-up
    30
    ·
    7 days ago

    This is still over engineered. Just connect directly to the database from the client instead of having an API endpoint.

    • Zos_Kia@lemmynsfw.com
      link
      fedilink
      arrow-up
      1
      ·
      5 days ago

      Supabase unironically do exactly that. You’ve got your client, you login through OAuth, then use your JWT to connect directly to postgres.

      Your JWT contains your user id, which is used with row level security rules to determine which rows you can and cannot access. It’s pretty amazing what you can do with PG alone. The tooling is not quite there yet but that’s probably where we’re headed.

  • Buckshot@programming.dev
    link
    fedilink
    arrow-up
    15
    ·
    6 days ago

    I got dumped with fixing some bugs in a project written by a contractor who had literally done this but with extra steps.

    Backend was sql server and c#/asp.

    There was an api endpoint that took json, used xslt to transform to xml. Then called the stored procedure specified in request passing the xml as a parameter.

    The stored procedure then queried the xml for parameters, executed the query, and returned results as xml.

    Another xslt transformed that to json and returned to the client.

    It was impressive how little c# there was.

    Despite holding all the business logic, the sql was not in source control.

    • lordbritishbusiness@lemmy.world
      link
      fedilink
      English
      arrow-up
      1
      ·
      5 days ago

      Worst thing with databases. Used to quote to my interns, “No spooky action at a distance” logic has to be in front of you and in git. Anything else is a recipe for bugs and undetectable errors.

      • Buckshot@programming.dev
        link
        fedilink
        arrow-up
        2
        ·
        5 days ago

        Yeah, maybe the contractor thought he’d get more work fixing it but he was long gone by the time I got it so i never met him

        One of bugs I got was performance because the search didn’t work, with about 600,000 assets in database it would timeout searching for one by exact match on ID. It took 45 minutes to return 1 result.

  • julianwgs@discuss.tchncs.de
    link
    fedilink
    arrow-up
    13
    arrow-down
    2
    ·
    6 days ago

    Why not?

    We did that for a Plotly dashboard in Python. We copied the database into a read-only in-memory sqlite database (it is quite small, only a couple thousand entries) to prevent any damages outside the dashboard. The data only gets updated every couple of days. You could skip this step. Then with sqlite you can restrict what action a query can use (SELECT, JSON, etc.) and you can restrict the instructions per query to prevent denial of service. It works like a charm and is much simpler than providing a REST API. Also the user might already know SQL.

    I am actually planning something similar for a task management web app I am building at the moment (additionally to providing a REST API). No need to learn another query language like in Jira.

    • shoo@lemmy.world
      link
      fedilink
      arrow-up
      20
      ·
      6 days ago

      Couple of reasons of varying importance:

      • Security. Even when you limit operations or table access it’s very easy to mess something up. Some new employee starts storing sensitive data in the wrong place or a db admin accidentally turns off the wrong permissions, etc…
      • It’s secretly more overengineered than a standard api despite looking simpler. If your app needs extremely robust query capabilities then you probably have a use case for an entire analytics stack and could use an open source option. Otherwise your users probably just need basic search, filtering, sorting, etc…
      • Ungodly, Flex Tape tier tight coupling. Part of the purpose of an api is to abstract away implementation details and present a stable contract. Now if you want to migrate/upgrade the database or add a new data source, everyone has to know about it and it’s potentially a major breaking change.
      • Familiarity. If someone else steps in to maintain it it’s much easier to get up to speed with a more standard stack. You don’t need a seven layer salad of enterprise abstraction bullshit, but it’s useful to see a familiar separation of auth, queries, security, etc…
      • Having the option to do business logic outside of the database can save countless headaches. Instead of inventing views or kludging sprocs to do some standard transformation, you can pull in a mature library. Some things, such as scrubbing PII, are probably damn near impossible without a higher tier layer to work in.
      • Client support. Your browser/device probably has a few billion options for consuming a REST/HATEOAS/graphql/whatever api. I doubt there’s many direct sql options with wide support.

      I probably wouldn’t do it outside of a tiny solo project. There are plenty of frameworks which do similar things (such as db driven apis) without compromising on flexibility, security or features.

  • Blackmist@feddit.uk
    link
    fedilink
    English
    arrow-up
    6
    ·
    6 days ago

    I did this just to reduce network latency. It’s not for public use, and tbh, I don’t think you can even get at it from outside the VPN.

      • Blackmist@feddit.uk
        link
        fedilink
        English
        arrow-up
        1
        ·
        5 days ago

        I haven’t been down to test their public wifi in the cafe to see if that can access it.

        The guy who installed it used to work for us and is a known clown, so it’s entirely possible.

        Although if it is, there’s way worse things they can do from there. Like connect to the actual database for a start.

        • luciferofastora@feddit.org
          link
          fedilink
          arrow-up
          1
          ·
          4 days ago

          Does the database use the same authentication and permissions as the API? If the API authenticates against the DB with a technical user, it may be still be an exploitable vulnerability for people who can’t access the DB directly but can access the API. I don’t know what database it is, what other databases run on the same server and what privileges might be achievable or escalatable, but generally “there are worse weaknesses” isn’t a solid security policy.

          You could give me a VPN access and I’ll take a look around :p

          (Please don’t, actually – in case it needs to be said, running pentests on prod is a dangerously bad idea already even before we get to the whole “trusting a stranger on the Internet just because they sound sorta knowledgeable” issue)

  • kibiz0r@midwest.social
    link
    fedilink
    English
    arrow-up
    10
    ·
    7 days ago

    Does ReST mean anything anymore? It was originally a set of principles guiding the development of the HTTP 1.1 spec. Then it meant mapping CRUD to HTTP verbs so application-agnostic load balancers could work right. And now I guess it’s just HTTP+JSON?

  • katy ✨@piefed.blahaj.zone
    link
    fedilink
    English
    arrow-up
    8
    ·
    7 days ago

    also stop putting in extra work to handle queries; the user knows what they want so let them enter the queries themselves and save development time. database sanitization is just pointless busywork.

  • kolorafa@lemmy.world
    link
    fedilink
    arrow-up
    8
    ·
    7 days ago

    It’s not that bad that you might think, the db user just need to have readonly access permissions to specific database tables.

    Ofc all data in tables ofc need to be public, so more like simple public facing page, app should not have any notion of users in any way, data probably populated by some automated system, and UI just to make it easier for anonymouse users to view that data in a friendly way.

    On top of that it will be a hell for the sysops as they will need to know the whole db structure and such of even a single part of db would contain non-public data, but that overall the best guy to handle security in the first place.

    And because all data is totally public in the first place you could give the task of creating frontend to any junior or LLM and it will be still secure.


    But in truth it is very bad idea (even it it is possible), because most likely the database connections would be reused so you could for example change current connection session timezone or other params and that ofc would nit change the data in db but still could affect other users by showing wrongly formatted or shifted data.

    • marcos@lemmy.world
      link
      fedilink
      arrow-up
      5
      ·
      7 days ago

      Most DBs have some way to reset reused connections. Postgres is one of those.

      The actual problem, even with public data, is that it’s trivial to overload a database with bad queries.

    • entwine@programming.dev
      link
      fedilink
      arrow-up
      4
      ·
      7 days ago

      CouchDB (a no-sql db, but whatever) automatically provides a REST API that’s designed to be exposed directly to clients. It even implements its own client-facing authentication system. “queries” are configured in advance from the admin side, and clients just pull the results, allowing for very efficient caching. Basically, if you RTFM enough to get a couchdb instance running, you have 90%-100% of your backend complete. You could create an entire scalable full-stack app using only client-side code… and if you’re clever with HTMX, you might even be able to do it without writing any javascript at all! (I tried once, but failed because I’m not that clever, but it’s definitely probably possible)

      So TL;DR: I like couchdb, and the idea of exposing your database directly to users isn’t unprecedented. I wonder if there are any SQL databases that offer a similar thing?

    • lordbritishbusiness@lemmy.world
      link
      fedilink
      English
      arrow-up
      2
      ·
      5 days ago

      Yeah, if I heard a junior dev say that in earshot I’ll be pulling them into a room for the list-of-reasons-that’s-dumb. Even the AI code assistants would directly say it’s not advisable.

  • TrickDacy@lemmy.world
    link
    fedilink
    arrow-up
    5
    ·
    7 days ago

    I wish I could go back to rest apis. My company is all in on graphql and it fucking sucks so much ass.