Skip to content

Make only authorized users be able to create a connection #159

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
Open
majecty opened this issue Feb 12, 2020 · 2 comments
Open

Make only authorized users be able to create a connection #159

majecty opened this issue Feb 12, 2020 · 2 comments
Labels
ics Interchain standard

Comments

@majecty
Copy link
Contributor

majecty commented Feb 12, 2020 •
edited
Loading

In the ICS connection spec, they are saying, "Each chain has knowledge of and has agreed to its identifier on the other chain."

We should prohibit malicious users from using reserved identifiers for other purposes.

We might need some authorization logic to make only allowed users be able to create connections.

Actions need authorization

  • CreateClient
  • ConnOpenInit
  • ChanOpenInit
@majecty majecty added the ics Interchain standard label Feb 12, 2020
@kseo kseo added the experiment Experimental features label Feb 27, 2020
@ScarletBlue ScarletBlue changed the title Make only authorized user be able to create a connection Make only authorized users be able to create a connection Mar 2, 2020
@majecty
Copy link
Contributor Author

majecty commented Mar 18, 2020

This issue is meaningful in the final version of ICS implementation (not only PoC implementation).
I'll remove the "experiment" tag.

@majecty majecty removed the experiment Experimental features label Mar 18, 2020
@majecty
Copy link
Contributor Author

majecty commented Mar 18, 2020

We may create a voting system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ics Interchain standard
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@majecty @kseo