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

Make atnotification->decrypted_value point to atnotification->value when it is a non-encrypted notification #559

Open
JeremyTubongbanua opened this issue Feb 19, 2025 · 0 comments
Assignees

Comments

@JeremyTubongbanua
Copy link
Member

The way we currently handle non-encrypted notifications in monitor is that monitor_read populates atnotification->value to store the value, since no decryption is needed. Therefore, atnotification->decrypted_value remains null.

The proposed change is to have atnotification->decrypted_value point to atnotification->value, since it is non-encrypted anyways. This is a developer experience change.

@XavierChanth XavierChanth self-assigned this Mar 2, 2025
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

No branches or pull requests

2 participants