• passepartout@feddit.org
      link
      fedilink
      arrow-up
      125
      ·
      8 days ago

      My friend who helped me research the OAuth vulnerabilities was let go for “security concerns from corporate”

      Good old shooting the messenger.

      • ZoteTheMighty@lemmy.zip
        link
        fedilink
        arrow-up
        16
        arrow-down
        2
        ·
        8 days ago

        I mean, they were an employee who was exploring security vulnerabilities with a non-employee who has a blog. I would have fired them too.

        • passepartout@feddit.org
          link
          fedilink
          arrow-up
          16
          ·
          8 days ago

          It is indeed a very risky move without a lot to gain for him personally. But I could guess McDonald’s would have forced him to ignore it and shut up about it if he disclosed this to the higher ups himself, in which case I would have gladly left myself instead.

  • Iced Raktajino@startrek.website
    link
    fedilink
    arrow-up
    35
    ·
    8 days ago

    I work with several people who would think this is a good idea.

    When they push it to prod, and our WAF goes 403 on every request, then suddenly it’s my problem to “fix”. Eye Roll

  • x00z@lemmy.world
    link
    fedilink
    English
    arrow-up
    30
    ·
    8 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
      ·
      6 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.

  • troed@fedia.io
    link
    fedilink
    arrow-up
    16
    ·
    8 days ago

    Great idea. How can we submit this to all AI scrapers?

    /cybersec red teamer

  • Buckshot@programming.dev
    link
    fedilink
    arrow-up
    14
    ·
    7 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
      ·
      6 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
        ·
        6 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
    ·
    7 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
      ·
      7 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.

  • kibiz0r@midwest.social
    link
    fedilink
    English
    arrow-up
    10
    ·
    8 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?

  • ozoned@piefed.social
    link
    fedilink
    English
    arrow-up
    9
    ·
    8 days ago

    I’m 100% for this. Because we’ll find out VERY quickly what sites are dumb enough to do this. And then we know to stay away from them! :-D

  • kolorafa@lemmy.world
    link
    fedilink
    arrow-up
    8
    ·
    8 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
      ·
      8 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
      ·
      8 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?