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-UI] Cloud onboarding token #198444

Merged
merged 21 commits into from
Nov 19, 2024
Merged

Conversation

XavierM
Copy link
Contributor

@XavierM XavierM commented Oct 30, 2024

Summary

The solution must be aware of the onboarding token from the cloud onboarding flow. With this information, it can redirect our users to the appropriate onboarding flow in Kibana based on their token. We need to create an API in kibana for cloud to save some basic data.

Checklist

@XavierM XavierM added Team:Cloud release_note:skip Skip the PR/issue when compiling release notes v9.0.0 v8.17.0 labels Oct 30, 2024
@XavierM XavierM marked this pull request as ready for review November 5, 2024 15:08
@XavierM XavierM requested review from a team as code owners November 5, 2024 15:08
@XavierM XavierM added the backport:version Backport to applied version labels label Nov 5, 2024
@ymao1 ymao1 removed the request for review from a team November 5, 2024 19:19
@afharo afharo added the Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! label Nov 6, 2024
@elasticmachine
Copy link
Contributor

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

Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

Tentatively approving on the condition that we don't risk leaking sensitive data to unauthorized route responses.

*/

import { httpServerMock, httpServiceMock } from '@kbn/core/server/mocks';
import { setDefaultSpaceSolutionType } from './set_cloud_data_route';
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be setPostCloudSolutionDataRoute?

.send({
onboardingData: {
solutionType: 'search',
token: 'connectors',
Copy link
Contributor

Choose a reason for hiding this comment

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

how confident are we that token won't expose sensitive data?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

we are super confident ;)

Copy link
Contributor

@TinaHeiligers TinaHeiligers left a comment

Choose a reason for hiding this comment

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

Actually LGTM

@XavierM XavierM enabled auto-merge (squash) November 19, 2024 21:59
@XavierM XavierM merged commit 158a828 into elastic:main Nov 19, 2024
23 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/11923785612

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Nov 19, 2024
## Summary

The solution must be aware of the onboarding token from the cloud
onboarding flow. With this information, it can redirect our users to the
appropriate onboarding flow in Kibana based on their token. We need to
create an API in kibana for cloud to save some basic data.

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Christiane (Tina) Heiligers <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
(cherry picked from commit 158a828)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Nov 20, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [[CLOUD-UI] Cloud onboarding token
(#198444)](#198444)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Xavier
Mouligneau","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-19T23:42:09Z","message":"[CLOUD-UI]
Cloud onboarding token (#198444)\n\n## Summary\r\n\r\nThe solution must
be aware of the onboarding token from the cloud\r\nonboarding flow. With
this information, it can redirect our users to the\r\nappropriate
onboarding flow in Kibana based on their token. We need to\r\ncreate an
API in kibana for cloud to save some basic data.\r\n\r\n###
Checklist\r\n\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: Christiane (Tina)
Heiligers <[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"158a828a8e4cf78390fa8711666609c8e16a71ca","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Security","Team:Cloud","release_note:skip","v9.0.0","backport:version","v8.17.0"],"title":"[CLOUD-UI]
Cloud onboarding
token","number":198444,"url":"https://github.com/elastic/kibana/pull/198444","mergeCommit":{"message":"[CLOUD-UI]
Cloud onboarding token (#198444)\n\n## Summary\r\n\r\nThe solution must
be aware of the onboarding token from the cloud\r\nonboarding flow. With
this information, it can redirect our users to the\r\nappropriate
onboarding flow in Kibana based on their token. We need to\r\ncreate an
API in kibana for cloud to save some basic data.\r\n\r\n###
Checklist\r\n\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: Christiane (Tina)
Heiligers <[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"158a828a8e4cf78390fa8711666609c8e16a71ca"}},"sourceBranch":"main","suggestedTargetBranches":["8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/198444","number":198444,"mergeCommit":{"message":"[CLOUD-UI]
Cloud onboarding token (#198444)\n\n## Summary\r\n\r\nThe solution must
be aware of the onboarding token from the cloud\r\nonboarding flow. With
this information, it can redirect our users to the\r\nappropriate
onboarding flow in Kibana based on their token. We need to\r\ncreate an
API in kibana for cloud to save some basic data.\r\n\r\n###
Checklist\r\n\r\n\r\n- [x] [Unit or
functional\r\ntests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)\r\nwere
updated or added to match the most common
scenarios\r\n\r\n---------\r\n\r\nCo-authored-by: Christiane (Tina)
Heiligers <[email protected]>\r\nCo-authored-by:
kibanamachine
<[email protected]>","sha":"158a828a8e4cf78390fa8711666609c8e16a71ca"}},{"branch":"8.x","label":"v8.17.0","branchLabelMappingKey":"^v8.17.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Xavier Mouligneau <[email protected]>
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Nov 26, 2024
## Summary

The solution must be aware of the onboarding token from the cloud
onboarding flow. With this information, it can redirect our users to the
appropriate onboarding flow in Kibana based on their token. We need to
create an API in kibana for cloud to save some basic data.

### Checklist


- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Christiane (Tina) Heiligers <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## Summary

The solution must be aware of the onboarding token from the cloud
onboarding flow. With this information, it can redirect our users to the
appropriate onboarding flow in Kibana based on their token. We need to
create an API in kibana for cloud to save some basic data.

### Checklist


- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

---------

Co-authored-by: Christiane (Tina) Heiligers <[email protected]>
Co-authored-by: kibanamachine <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:version Backport to applied version labels release_note:skip Skip the PR/issue when compiling release notes Team:Cloud Team:Security Team focused on: Auth, Users, Roles, Spaces, Audit Logging, and more! v8.17.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants