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

Remove authz messages #823

Merged
merged 1 commit into from
Aug 26, 2024
Merged

Remove authz messages #823

merged 1 commit into from
Aug 26, 2024

Conversation

lubkoll
Copy link
Contributor

@lubkoll lubkoll commented Aug 26, 2024

No description provided.

magiodev
magiodev approved these changes Aug 26, 2024
View reviewed changes
Copy link
Contributor

@magiodev magiodev left a comment

Choose a reason for hiding this comment

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

changes look good to me. please ensure that we internally confirm whether this implementation was done for a specific reason or to support any particular consumer. this has been kept unused so far afaik tho

@lubkoll lubkoll merged commit 808dc27 into main Aug 26, 2024
3 checks passed
@lubkoll lubkoll deleted the feat/rm-authz branch August 26, 2024 22:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@magiodev magiodev magiodev approved these changes

@0xLaurenzo 0xLaurenzo Awaiting requested review from 0xLaurenzo

@arhamchordia arhamchordia Awaiting requested review from arhamchordia

@ajansari95 ajansari95 Awaiting requested review from ajansari95

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lubkoll @magiodev