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

remove capability authentication checks in core handlers that are routed from ICS 26 #983

Conversation

crodriguezvega
Copy link
Contributor

@crodriguezvega crodriguezvega commented Jun 12, 2023

closes: #886

I have also updated all the onChanOpenInit and onChanOpenTry callbacks to receive the channel capability as an argument and adapted the implementation of the callbacks to claim the channel capability where required.

@colin-axner
Copy link
Contributor

Is this r4r?

@crodriguezvega crodriguezvega marked this pull request as ready for review June 23, 2023 09:02
Carlos Rodriguez added 2 commits October 2, 2023 13:41
@crodriguezvega
Copy link
Contributor Author

I will close this PR, but keep the branch for now. I might be able to reuse it for #1133.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Remove port/channel capability authentication from ics 026 routed handlers
2 participants