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

SALTO-4116: Support fetch and deploy for provisioning in apps #7003

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

idoamit8
Copy link
Contributor

@idoamit8 idoamit8 commented Dec 26, 2024

In this PR I support fetch and deploy for provisioning in apps


Additional context for reviewer
Some public APIs support User Provisioning (referred to as "ToApp" in the UI) and Inbound Provisioning (referred to as "To Okta" in the UI). These endpoints currently support a limited number of applications, as listed below:
match: ['^google$', '^office365$', '^okta_org2org$', '^slack$', '^zoomus$', '^zscalerbyz$']
Additionally, all applications have a private API that supports the To Okta page.
I plan to use the public API for the supported applications to enable both fetching and deploying their provisioning. For the remaining applications, I will support fetching and deploying only for what is represented by To Okta.

After first review I will add:
1. Tests
2. CV to say that we are not support enablement of provisioning (Although we will be able to support it if we turn off the CV (but its a mature of authentication)

Some screenshots:

applicationUserProvisioning applicationInboundProvisioning applicationProvisioningGeneral applicationProvisioningUsers
image image image image

Release Notes:
Replace me with a short sentence that describes the effect of this change on Salto users


User Notifications:
Replace me with a short sentence that describes changes that will appear in NaCls and are not caused by user actions (e.g. a new annotation, field values that are converted to references, etc). Hidden changes should not be listed.

@idoamit8 idoamit8 force-pushed the SALTO-4116-SupportProvisioningForApps branch from 2d382f8 to 2ded985 Compare December 26, 2024 08:54
@coveralls
Copy link

coveralls commented Dec 26, 2024

Coverage Status

coverage: 93.651% (-0.001%) from 93.652%
when pulling f1d38e5 on idoamit8:SALTO-4116-SupportProvisioningForApps
into 0e9be61 on salto-io:main.

@idoamit8 idoamit8 force-pushed the SALTO-4116-SupportProvisioningForApps branch from 2ded985 to 9e86d66 Compare December 26, 2024 11:28
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