fix: get notifications not authorized #305

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

Draft

chris13524 wants to merge 1 commit into main from fix/get-notifications-not-authorized

Member

chris13524 commented Jan 20, 2024

Description

Fixes that the authorization for get notifications isn't checked to see that the account matches the subscription.

How Has This Been Tested?

TODO

Due Diligence

Breaking change
Requires a documentation update
Requires a e2e/integration test update

chris13524 self-assigned this


          fix: get notifications not authorized

5d694c7

chris13524 force-pushed the fix/get-notifications-not-authorized branch from 855bf9c to 5d694c7 Compare

January 20, 2024 16:10

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

Labels

None yet