Skip to content

feat: add --permissionless flag for authorization commands#556

Open
PivasDesant wants to merge 22 commits intoPsycheFoundation:mainfrom
PivasDesant:feat/add-permissionless-flag
Open

feat: add --permissionless flag for authorization commands#556
PivasDesant wants to merge 22 commits intoPsycheFoundation:mainfrom
PivasDesant:feat/add-permissionless-flag

Conversation

@PivasDesant
Copy link

Adds --permissionless flag to authorization commands as an alternative to --authorizer 11111111111111111111111111111111.

The flag automatically uses system_program::ID internally and conflicts with --authorizer to prevent misuse.

Affected commands:

  • join-authorization-create
  • join-authorization-delete
  • join-authorization-read
  • can-join

Closes #471

@arilotter
Copy link
Collaborator

Thank you for the PR!!! this is sweet, almost exactly what i need. now that i see it down in a PR, I wonder if we shouldn't generally replace the authorization stuff in the CLIs with an enum, like

enum Authorization {
  Address(..),
  Permissionless
 }

and then can implement this in such a way that if you pass --authorization permissionless that maps to the 11111111 addr.... @crypto-vincent what do you think?

@crypto-vincent
Copy link
Contributor

crypto-vincent commented Feb 15, 2026

Thank you for the PR!!! this is sweet, almost exactly what i need. now that i see it down in a PR, I wonder if we shouldn't generally replace the authorization stuff in the CLIs with an enum, like

enum Authorization {
  Address(..),
  Permissionless
 }

and then can implement this in such a way that if you pass --authorization permissionless that maps to the 11111111 addr.... @crypto-vincent what do you think?

Thanks everyone for looking into this! I'm also wondering if there's a way to implement this without using the conflict_with type of logic on the CLI. Maybe --authorization NONE or something, but i'm not sure what would be the idiomatic thing to do here.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Permissionless authorization flag

3 participants