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

Shall we add a new resource for <Role> #573

Open
isc-shuliu opened this issue Sep 19, 2024 · 4 comments
Open

Shall we add a new resource for <Role> #573

isc-shuliu opened this issue Sep 19, 2024 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@isc-shuliu
Copy link
Collaborator

See #562 (comment) and #572 (comment)

Shall we add a new resource <Role> in module.xml?

According to the online doc, it can look like

<Role 
  Name="MyRoleName"
  Descrpition="my role description"
  Resources="MyResource1:RW,MyResource2:RWU"
  GrantedRoles="Role1,Role2"
/>

Thoughts: Maybe change Resources to RoleResources to be distinguished from IPM Resources in module.xml

@isc-shuliu isc-shuliu self-assigned this Sep 19, 2024
@isc-shuliu isc-shuliu added the enhancement New feature or request label Sep 19, 2024
@isc-shuliu
Copy link
Collaborator Author

@isc-tleavitt @isc-kiyer Any thoughts?

@isc-kiyer
Copy link
Collaborator

@isc-shuliu Hmm if we go down this route we are basically exposing stuff that a customer can do with a cpf merge file. I wonder if it would be better to instead just add IPM support for cpf merge files instead of re-implementing wrappers for the various Config.* and Security.* classes?
It may be a bit tricky though since the iris merge command needs to be run from the OS and I'm not sure if $zf(-100) can run it: https://docs.intersystems.com/iris20242/csp/docbook/DocBook.UI.Page.cls?KEY=ACMF#ACMF_reconfig
cc @isc-tleavitt

@SCanzano
Copy link

I found myself here as I was going to submit a request that is similar to this, My specific request is that I want to be able to define in a Module definition the name of Security Resources that should be installed. I know I can do this with a custom
but it seems like many of us would want to be able to add with the installation of a module

  • Security Resources
  • Security Roles
  • Security Users

@isc-tleavitt
Copy link
Contributor

@SCanzano Resources and Roles make sense, but Users seem a bit more risky. Could you elaborate on the use case for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants