Skip to content

Conversation

LDiazN
Copy link
Contributor

@LDiazN LDiazN commented Oct 6, 2025

This PR will change the ooniauth-py API to use strings instead of byte arrays to simplify usage in Python land

Encoding of binary data structures into string is made with base64 encoding

closes #11

@LDiazN LDiazN requested a review from hellais October 6, 2025 10:49
@LDiazN LDiazN self-assigned this Oct 6, 2025
LDiazN added 6 commits October 6, 2025 13:07
When compiling the library, the resulting .whl will only be
compatible against the same exact python version it was
compiled against.

By specifying an abi minimum compatibility we make the library
compatible with multiple python version.

abi3-py310 for example makes the resulting wheel compatible
with any python version higher than 3.10

This change is required to make the library easier to compile
and compatible with ooniprobe

See:
- https://pyo3.rs/main/features.html#abi3
- https://pyo3.rs/main/building-and-distribution#py_limited_apiabi3
Copy link
Collaborator

@mmaker mmaker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@LDiazN can you please lint this? I think type annotation require a space should be e.g. response: str

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

In the Python bindings, move string serialization to Rust land

2 participants