Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Crypto] Implement support for EncryptAlpha1/DecryptAlpha1 APIs #548

Open
ItalyPaleAle opened this issue Apr 5, 2023 · 2 comments · May be fixed by #719
Open

[Crypto] Implement support for EncryptAlpha1/DecryptAlpha1 APIs #548

ItalyPaleAle opened this issue Apr 5, 2023 · 2 comments · May be fixed by #719
Assignees
Labels
help wanted Extra attention is needed
Milestone

Comments

@ItalyPaleAle
Copy link
Contributor

dapr/dapr#6197 implements two new APIs in Dapr:

  • EncryptAlpha1
  • DecryptAlpha1

Both APIs are stream-based, so the SDK is expected to read input as a stream, send it to Dapr using a streamed gRPC in chunks, and in parallel read the response from Dapr (in chunks sent over the same gRPC stream) and return a readable stream to the caller.

The reference implementation is in the Go SDK: dapr/go-sdk#387

@berndverst
Copy link
Member

@ItalyPaleAle FYI in the Python SDK we don't name the functions Alpha1 etc. We use a stable and then point the implementation to the Alpha API, but add a warning that the API being used is Alpha.

@berndverst berndverst added the help wanted Extra attention is needed label Apr 11, 2023
@berndverst berndverst added this to the v1.11 milestone May 30, 2023
@berndverst berndverst modified the milestones: v1.11, v1.13 Oct 31, 2023
@berndverst berndverst modified the milestones: v1.13, v1.14 Feb 13, 2024
@KentHsu
Copy link
Contributor

KentHsu commented May 14, 2024

/assign

@KentHsu KentHsu linked a pull request May 24, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants