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.

Sign up for GitHub

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

Jump to bottom

Return unencrypted data #156

Merged
merged 3 commits into from
Nov 23, 2023
Merged

Return unencrypted data #156

merged 3 commits into from
Nov 23, 2023

Conversation

parfeon
Copy link
Contributor

@parfeon parfeon commented Nov 23, 2023 •
edited
Loading

refactor(crypto): return unencrypted data

Return source event data when a client configured with crypto is unable to decrypt it (can be not encrypted data or encrypted with different options)

@parfeon
refactor(crypto): return unencrypted data
b913081 
Return source event data when client configured with crypto is unable to decrypt it (can be not
encrypted data or encrypted with different options)
@parfeon parfeon added status: done This issue is considered resolved. priority: medium This PR should be reviewed after all high priority PRs. type: refactor This PR contains refactored existing features. labels Nov 23, 2023
@parfeon parfeon self-assigned this Nov 23, 2023
seba-aln
seba-aln approved these changes Nov 23, 2023
View reviewed changes
Copy link
Contributor

@seba-aln seba-aln left a comment

Choose a reason for hiding this comment

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

Indeed, young man. This work exhibits a commendable blend of ingenuity and diligence, a testament to your admirable commitment to excellence in our company endeavors.

@parfeon
Copy link
Contributor Author

parfeon commented Nov 23, 2023

@pubnub-release-bot release

@parfeon parfeon merged commit 0acb1d8 into master Nov 23, 2023
7 checks passed
@parfeon parfeon deleted the CLEN-1706/refactor/crypto branch November 23, 2023 21:23
@pubnub-release-bot
Copy link

🚀 Release successfully completed 🚀

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

@seba-aln seba-aln seba-aln approved these changes

Assignees

@parfeon parfeon

Labels
priority: medium This PR should be reviewed after all high priority PRs. status: done This issue is considered resolved. type: refactor This PR contains refactored existing features.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@parfeon @pubnub-release-bot @seba-aln