Re: encode/decode support for base64url

From: Florents Tselai <florents(dot)tselai(at)gmail(dot)com>
To: Daniel Gustafsson <daniel(at)yesql(dot)se>
Cc: "David E(dot) Wheeler" <david(at)justatheory(dot)com>, Aleksander Alekseev <aleksander(at)tigerdata(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Cary Huang <cary(dot)huang(at)highgo(dot)ca>, Przemysław Sztoch <przemyslaw(at)sztoch(dot)pl>
Subject: Re: encode/decode support for base64url
Date: 2025-08-01 10:13:23
Message-ID: CA+v5N428jbA50S_R4BY=jg2WFPjTjm5rM4JnyE1_5==91ogTDA@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Jul 29, 2025 at 3:25 PM Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:

> > On 12 Jul 2025, at 21:40, David E. Wheeler <david(at)justatheory(dot)com>
> wrote:
>
> > Thank you! This looks great. The attached revision makes a a couple of
> minor changes:
>
> I also had a look at this today and agree that it looks pretty close to
> being
> done, and a feature we IMHO would like to have.

Thanks for having a look Daniel!

>
>
The attached version also adds a commit message, tweaks the documentation
> along
> with a few small changes to error message handling etc.
>

In the doc snippet

> The base64url alphabet use '-' instead of '+' and '_' instead of '/' and
also omits the '=' padding character.

Should be

> The base64url alphabet use*s* '-' instead of '+' and '_' instead of '/'*,
*and also omits the '=' padding character.

I'd also add a comma before "and also"

> The base64 code this extends is the RFC 2045 variant while base64url is
> based
> on base64 from RFC 3548 (obsoleted by RFC 4648). AFAICT this is not a
> problem
> here but has anyone else verified this?
>

I don't see how this can be a problem in practice.
The conversions are straightforward,
and the codepath used with url=true is a new one and doesn't change past
behavior.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Dilip Kumar 2025-08-01 10:28:34 Re: Conflict detection for update_deleted in logical replication
Previous Message Peter Eisentraut 2025-08-01 10:03:28 Re: C11 / VS 2019