Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use BASE64_MIME_PERMISSIVE to allow trailing bits in b64 data #125

Closed
wants to merge 0 commits into from

Conversation

wathiede
Copy link
Contributor

@wathiede wathiede commented Apr 9, 2024

This should address #96

Note, this requires a not yet released https://github.com/ia0/data-encoding and shouldn't be submitted until data-encoding v2.6.0 is released. Please let me know if there's a better way to deal with an upstream feature that is currently only in git (for example, if you want me to update the Cargo.toml to depend on a git repo instead of the crate).

Alos, I'm happy to wait until v2.6.0 is released.

@staktrace
Copy link
Owner

Thanks for the PR! I'd like to wait until the new version of data encoding is published before merging this. I'll try to check periodically but if you ping here once it's published that would be great and I can merge this.

@wathiede
Copy link
Contributor Author

With ia0/data-encoding#104 version 2.6.0 of data-encoding has been published to https://crates.io/crates/data-encoding

I believe this PR is now safety mergable.

@staktrace
Copy link
Owner

Ah oops I accidentally closed closed this by pushing the wrong branch. I just wanted to squash the commits. Hang on a sec

@staktrace
Copy link
Owner

Let's try this...

@staktrace
Copy link
Owner

Ok I just made a new PR, #126 with the squashed commit. I've retained your authorship on the commit there.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants