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

[Cloud Security] Add license gate to integration page #189855

Closed
JordanSh opened this issue Aug 4, 2024 · 5 comments · Fixed by #190285
Closed

[Cloud Security] Add license gate to integration page #189855

JordanSh opened this issue Aug 4, 2024 · 5 comments · Fixed by #190285
Assignees
Labels
8.16 candidate Feature:Cloud-Security Cloud Security related features Team:Cloud Security Cloud Security team related

Comments

@JordanSh
Copy link
Contributor

JordanSh commented Aug 4, 2024

Summary

We want to move our license checks from the findings and dashboard page to the integration page so 3P can use our pages without being blocked in case the user have a lower license than what is required for our integration

DOD

  • According to the design, integration page should either be blocked or installation blocked
  • Block the installation from the API (might already be implemented, need to check)
  • Tests as needed

Related

@JordanSh JordanSh added the Team:Cloud Security Cloud Security team related label Aug 4, 2024
@JordanSh JordanSh self-assigned this Aug 4, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-cloud-security-posture (Team:Cloud Security)

@JordanSh JordanSh added Feature:Cloud-Security Cloud Security related features 8.16 candidate labels Aug 4, 2024
@kfirpeled
Copy link
Contributor

This is already implemented in our API - moved release priority to nice to have

@kfirpeled
Copy link
Contributor

Checkout the implications of changing our integration subscription to enterprise

It looks like the package-spec supports different subscriptions: https://github.com/elastic/package-spec/blob/d4119e18552647abe31b6dc662ba1bd6c8e43e33/spec/integration/manifest.spec.yml#L106

@kfirpeled
Copy link
Contributor

kfirpeled commented Sep 11, 2024

Cant be tested on serverless - relevant only for on-prem

@opauloh
Copy link
Contributor

opauloh commented Nov 4, 2024

Cant be tested on ESS - relevant only for on-prem

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
8.16 candidate Feature:Cloud-Security Cloud Security related features Team:Cloud Security Cloud Security team related
Projects
None yet
4 participants