98
React2Shell: maximum severity vulnerability discovered in React (www.tenable.com)
submitted 11 months ago by to c/programming@programming.dev
8 comments fedilink hide all child comments
 

I got an email from Vercel urging to upgrade Next.js based project 3 days ago. POC was published 2 days ago. Today I've checked my logs and I could already see attack attempts.

you are viewing a single comment's thread
view the rest of the comments
[–] 11 points 11 months ago (5 children)

This only affects sites that use the fancy new-fangled SSR rendering from what I've read. ReactJS (Client side) sites wouldn't be affected.

  • permalink
  • fedilink
  • source
  • hideshow 5 child comments
    •
  • [–] 8 points 11 months ago (3 children)

    Are we still vulnerable if our app doesn’t use React Server Functions endpoints?

    Potentially. According to the React Team, even if React Server Functions are not in-use, the vulnerability is still exploitable if React Server Components are supported.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 3 child comments
    •
  • [–] 6 points 11 months ago (2 children)

    I think their point was that CSR-only sites would be unaffected, which should be true. Exploiting it on a static site, for example, couldn't be RCE because the untrusted code is only being executed on the client side (and therefore is not remote).

    Now, most people use, or at least are recommended to use, SSR/RSC these days. Many frameworks make SSR enabled by default. But using raw React with no Next.js, react-router, etc. to create a client-side only site does likely protect you from this vulnerability.

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 2 child comments
    •
  • [–] 4 points 11 months ago (1 child)

    Now, most people use, or at least are recommended to use, SSR/RSC these days.

    Why though?

  • permalink
  • fedilink
  • source
  • parent
  • hideshow 1 child comment
    •
  • [–] 2 points 11 months ago

    I'm not the one recommending it lol.

    If I had to guess, it's to improve page performance by prerendering as much as possible, but I find it overkill and prefer to just prerender as much of the page as I can at build time and do CSR for the rest, though this doesn't work if you have dynamic routes or some kind of server-side logic (good for blogs and such though).

  • permalink
  • fedilink
  • source
  • parent
    • load more comments (1 reply)