Hi,

Wido den Hollander wrote:
> With URL-safe base64 the + and / are replaced by - and _
> 
> This way we can accept base64 strings which are URL-safe.

There are many more variants of base64, see
  http://en.wikipedia.org/wiki/Base64

The current code is meant to implement RFC 4648. Since for RFC 4648 characters
outside the alphabet are forbidden, your patch would break this use-case.

I think if the module should be usable for other variants than RFC 4648,
the proper implementation is to have an 'enum' that identifies the variant
and to pass this enum as parameter to the functions isbase64, base64_encode,
base64_decode_ctx, etc.

Bruno


Reply via email to