Re: encode/decode support for base64url

Florents Tselai <florents.tselai@gmail.com>

From: Florents Tselai <florents.tselai@gmail.com>
To: Aleksander Alekseev <aleksander@tigerdata.com>
Cc: PostgreSQL Hackers <pgsql-hackers@postgresql.org>, Cary Huang <cary.huang@highgo.ca>, Daniel Gustafsson <daniel@yesql.se>, Przemysław Sztoch <przemyslaw@sztoch.pl>
Date: 2025-07-10T03:25:29Z
Lists: pgsql-hackers

Attachments

Thanks for the review Aleksander,

> On 9 Jul 2025, at 10:45 PM, Aleksander Alekseev <aleksander@tigerdata.com> wrote:
> 
> Hi Florents,
> 
> Thanks for the update!
> 
>> here's a v4 patch set
>> 
>> - Extracted pg_base64_{en,de}_internal with an  additional bool url param, to be used by other functions
>> - Added a few more test cases
>> 
>> Cary mentioned above
>> 
>>> In addition, you may also want to add the C versions of base64rul encode
>> and decode functions to "src/common/base64.c" as new API calls
>> 
>> Haven't done that, but I could;
>> Although I think it'd probably be best to do it in a separate patch.
> 
> I reviewed and tested v4. To me it looks as good as it will get.
> Personally I would change a few minor things here and there and
> probably merge all three patches into a single commit. This however is
> up to the committer to decide.

Attaching a single-file patch