Re: [PATCH] oauth: Prevent stack overflow by limiting JSON parse depth

From: Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com>
To: Aleksander Alekseev <aleksander(at)timescale(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] oauth: Prevent stack overflow by limiting JSON parse depth
Date: 2025-05-08 15:45:04
Message-ID: CAOYmi+kBCym6yT8pMJOiSQ-OciNGmbK0COyNS4jGPDX7ErMSRw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, May 8, 2025 at 5:22 AM Aleksander Alekseev
<aleksander(at)timescale(dot)com> wrote:
> Thanks for the patch. It looks good to me. It's well documented and
> covered with tests. I can confirm that the tests pass. Also they fail
> if I decrease the $nesting_limit value to 15.

Thanks for the review!

--Jacob

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Sami Imseih 2025-05-08 16:10:05 Re: Prolonged truncation phase during vacuum on toast table with repeated interruptions by lock waiters and a proposed POC patch
Previous Message Jacob Champion 2025-05-08 15:29:49 Re: [PATCH] Fix missing comma in Requires.private with a Make macro