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

Throw exception if token has no or multiple owners #118

Closed
Syndesi opened this issue Sep 11, 2023 · 0 comments · Fixed by #139
Closed

Throw exception if token has no or multiple owners #118

Syndesi opened this issue Sep 11, 2023 · 0 comments · Fixed by #139
Labels
Enhancement Improving existing functionalities. Security Addressing security-related tasks.
Milestone

Comments

@Syndesi
Copy link
Member

Syndesi commented Sep 11, 2023

Tokens must always have a single owner.

No owner is not possible, as the security models requires an user to be part of the ownership path. Similarly, multiple owners would likely cause troubles and "merged personalities".

@Syndesi Syndesi added this to the 0.1.0 milestone Sep 11, 2023
@Syndesi Syndesi changed the title Throw exception of token has no or multiple owners Throw exception if token has no or multiple owners Sep 11, 2023
Syndesi added a commit that referenced this issue Sep 26, 2023
- Add feature test to test that tokens not owned by any user are invalid, closes #118.
- Add feature test to test that tokens owned by two or more users are invalid as well, closes #118.
@Syndesi Syndesi mentioned this issue Sep 26, 2023
@Syndesi Syndesi added Enhancement Improving existing functionalities. Security Addressing security-related tasks. labels May 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Improving existing functionalities. Security Addressing security-related tasks.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant