Allow user defined encodings.

[rustyconover]

Add encode_with_encoding and decode_with_encoding which
allow the user to use pluggable encodings rather than just MIME::Base64.

Change encode and decode to just specify their encoding functions based
on MIME::Base64.

[dagolden]

What do you think about just extending encode and decode to take arguments and avoiding method calls to encode_with_encoding etc? E.g.

$store->encode( $data, $expires, $encoding_function )

where $encoding_function is set to the default if not defined?

David

[dagolden]

My next question is whether this really needs to be per-invocation of encode/decode or whether they should be properties of the object itself.

Could you please tell me a bit more about why you want this customization and what you're trying to do?

Thanks.

[rustyconover]

Hi David,

I'm using Session::Storage::Secure to create a string that is used as part of a custom reply to address for an email that I'm sending. When a user replies to that address, I decode the data that I stored in the address and then I know how to process their reply, rather than exposing that information in plain text in the email address or having a separate lookup table.

MIME::Base64 uses characters that aren't permissible in email addresses and it also distinguishes from upper and lower case. Whereas MIME::Base32 works perfectly for my needs and shares a very close to the same interface.

Thanks,

Rusty

[dagolden]

Makes sense. I'll implement this as attributes instead of as method arguments.

[rustyconover]

I’m happy to do that too if you want…

Rusty

-- 
Rusty Conover
Lucky Dinosaur, LLC.

On April 17, 2014 at 1:02:07 PM, David Golden (notifications@github.com) wrote:

Makes sense. I'll implement this as attributes instead of as method arguments.

—
Reply to this email directly or view it on GitHub.

[dagolden]

I'll do it. It's not a lot of work, but I might rename some stuff in the process.

[dagolden]

Shipped to CPAN. I also added the ability to change separator character, which you might also need.