Skip to content

feat: jh registry permission list, set and delete commands#28

Merged
thelonewolf1603 merged 13 commits intomainfrom
nd/registry-permission
Apr 17, 2026
Merged

feat: jh registry permission list, set and delete commands#28
thelonewolf1603 merged 13 commits intomainfrom
nd/registry-permission

Conversation

@thelonewolf1603
Copy link
Copy Markdown
Contributor

@thelonewolf1603 thelonewolf1603 commented Mar 6, 2026

PR also contains changes for user and groups list commands, divided into two cases

  • jh admin user list and jh admin group list - all users and groups with additional metadata (--verbose)

    • uses REST endpoints /app/config/features/manage and /app/config/groups respectively
  • jh user list and jh group list - relevant for registry sharing

    • uses public graphql endpoints

    fixes https://github.com/JuliaComputing/JuliaHub/issues/21363

Comment thread README.md
Comment on lines +158 to +161
- `jh registry config <name>` - Show the full JSON configuration for a registry
- `jh registry config add` - Add a new registry (JSON payload via stdin or `--file`)
- `jh registry config update` - Update an existing registry (same JSON schema as add, same flags)
- `jh registry permission list <registry>` - List permissions for a registry
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why are these changes part of this PR

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

same, checked out from the registry config pr, will undo after that is merged

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

Comment thread main.go
"type": "cacheserver", "host": "https://pkg.juliahub.com",
"credential_key": "JC Auth Token",
"server_type": "", "github_credential_type": "", "api_host": "", "url": "", "user_name": ""
"credential_key": "JC Auth Token"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Invallid json

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed, was carried over from prev pr

Comment thread main.go Outdated
"download_providers": [{
"type": "cacheserver", "host": "https://pkg-new.juliahub.com",
"credential_key": "JC Auth Token",
"server_type": "", "github_credential_type": "", "api_host": "", "url": "", "user_name": ""
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Invalid JSON

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

same

thelonewolf1603 and others added 3 commits April 1, 2026 07:50
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Comment thread README.md Outdated
Comment on lines +346 to +347
"credential_key": "JC Auth Token",
"server_type": "", "github_credential_type": "", "api_host": "", "url": "", "user_name": ""
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We have a duplicate key here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

Comment thread CLAUDE.md
"type": "cacheserver", "host": "https://pkg.juliahub.com",
"credential_key": "JC Auth Token",
"server_type": "", "github_credential_type": "", "api_host": "", "url": "", "user_name": ""
"credential_key": "JC Auth Token"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We have a duplicate key here

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

removed

@thelonewolf1603 thelonewolf1603 merged commit a6835d4 into main Apr 17, 2026
1 check passed
@thelonewolf1603 thelonewolf1603 deleted the nd/registry-permission branch April 17, 2026 05:53
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.

2 participants