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

added '\' escapes #199185

Merged
merged 1 commit into from
Nov 6, 2024
Merged

added '\' escapes #199185

merged 1 commit into from
Nov 6, 2024

Conversation

bryce-b
Copy link
Contributor

@bryce-b bryce-b commented Nov 6, 2024

Summary

This PR adds backslash escaping to previous code scan fixes.

@bryce-b bryce-b added backport:version Backport to applied version labels v7.17.26 v8.15.4 Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team labels Nov 6, 2024
@bryce-b
Copy link
Contributor Author

bryce-b commented Nov 6, 2024

Added targeted back-ports for 7.17 and 8.x. Hopefully I did that right.

@bryce-b bryce-b marked this pull request as ready for review November 6, 2024 17:13
@bryce-b bryce-b requested a review from a team as a code owner November 6, 2024 17:13
@elasticmachine
Copy link
Contributor

Pinging @elastic/obs-ux-infra_services-team (Team:obs-ux-infra_services)

@botelastic botelastic bot added the ci:project-deploy-observability Create an Observability project label Nov 6, 2024
Copy link
Contributor

github-actions bot commented Nov 6, 2024

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@jennypavlova jennypavlova added the release_note:skip Skip the PR/issue when compiling release notes label Nov 6, 2024
@bryce-b bryce-b enabled auto-merge (squash) November 6, 2024 18:14
@bryce-b bryce-b merged commit fb71142 into elastic:main Nov 6, 2024
35 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 7.17, 8.15

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

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
infra 1.7MB 1.7MB +22.0B

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

This PR adds backslash escaping to previous code scan fixes.

(cherry picked from commit fb71142)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
7.17
8.15 Backport failed because of merge conflicts

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

Manual backport

To create the backport manually run:

node scripts/backport --pr 199185

Questions ?

Please refer to the Backport tool documentation

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

This will backport the following commits from `main` to `7.17`:
- [added '\' escapes
(#199185)](#199185)

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

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

<!--BACKPORT [{"author":{"name":"Bryce
Buchanan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-06T19:02:41Z","message":"added
'\\' escapes (#199185)\n\n## Summary\r\n\r\nThis PR adds backslash
escaping to previous code scan
fixes.","sha":"fb7114222b5bde5846fdda85a2ae0e94797600e2","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","ci:project-deploy-observability","Team:obs-ux-infra_services","backport:version","v8.15.4","v7.17.26"],"title":"added
'\\'
escapes","number":199185,"url":"https://github.com/elastic/kibana/pull/199185","mergeCommit":{"message":"added
'\\' escapes (#199185)\n\n## Summary\r\n\r\nThis PR adds backslash
escaping to previous code scan
fixes.","sha":"fb7114222b5bde5846fdda85a2ae0e94797600e2"}},"sourceBranch":"main","suggestedTargetBranches":["8.15","7.17"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199185","number":199185,"mergeCommit":{"message":"added
'\\' escapes (#199185)\n\n## Summary\r\n\r\nThis PR adds backslash
escaping to previous code scan
fixes.","sha":"fb7114222b5bde5846fdda85a2ae0e94797600e2"}},{"branch":"8.15","label":"v8.15.4","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"7.17","label":"v7.17.26","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Bryce Buchanan <[email protected]>
mgadewoll pushed a commit to mgadewoll/kibana that referenced this pull request Nov 7, 2024
## Summary

This PR adds backslash escaping to previous code scan fixes.
@bryce-b
Copy link
Contributor Author

bryce-b commented Nov 7, 2024

💚 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

bryce-b added a commit to bryce-b/kibana that referenced this pull request Nov 7, 2024
## Summary

This PR adds backslash escaping to previous code scan fixes.

(cherry picked from commit fb71142)

# Conflicts:
#	x-pack/plugins/observability_solution/infra/public/pages/metrics/metrics_explorer/components/helpers/create_tsvb_link.ts
bryce-b added a commit that referenced this pull request Nov 21, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [added '\' escapes
(#199185)](#199185)

<!--- Backport version: 8.9.8 -->

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

<!--BACKPORT [{"author":{"name":"Bryce
Buchanan","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-06T19:02:41Z","message":"added
'\\' escapes (#199185)\n\n## Summary\r\n\r\nThis PR adds backslash
escaping to previous code scan
fixes.","sha":"fb7114222b5bde5846fdda85a2ae0e94797600e2","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","ci:project-deploy-observability","Team:obs-ux-infra_services","backport:version","v8.15.4","v7.17.26"],"number":199185,"url":"https://github.com/elastic/kibana/pull/199185","mergeCommit":{"message":"added
'\\' escapes (#199185)\n\n## Summary\r\n\r\nThis PR adds backslash
escaping to previous code scan
fixes.","sha":"fb7114222b5bde5846fdda85a2ae0e94797600e2"}},"sourceBranch":"main","suggestedTargetBranches":["8.15"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/199185","number":199185,"mergeCommit":{"message":"added
'\\' escapes (#199185)\n\n## Summary\r\n\r\nThis PR adds backslash
escaping to previous code scan
fixes.","sha":"fb7114222b5bde5846fdda85a2ae0e94797600e2"}},{"branch":"8.15","label":"v8.15.4","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"7.17","label":"v7.17.26","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/199213","number":199213,"state":"MERGED","mergeCommit":{"sha":"83ca0265b5d9a0e11dd8092c7f9ab76c0df00d56","message":"[7.17]
added &#x27;\\&#x27; escapes (#199185) (#199213)\n\n# Backport\n\nThis
will backport the following commits from `main` to `7.17`:\n- [added
&#x27;\\&#x27;
escapes\n(#199185)](https://github.com/elastic/kibana/pull/199185)\n\n<!---
Backport version: 9.4.3 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Bryce\nBuchanan\",\"email\":\"[email protected]\"},\"sourceCommit\":{\"committedDate\":\"2024-11-06T19:02:41Z\",\"message\":\"added\n'\\\\'
escapes (#199185)\\n\\n## Summary\\r\\n\\r\\nThis PR adds
backslash\nescaping to previous code
scan\nfixes.\",\"sha\":\"fb7114222b5bde5846fdda85a2ae0e94797600e2\",\"branchLabelMapping\":{\"^v9.0.0$\":\"main\",\"^v8.17.0$\":\"8.x\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"release_note:skip\",\"v9.0.0\",\"ci:project-deploy-observability\",\"Team:obs-ux-infra_services\",\"backport:version\",\"v8.15.4\",\"v7.17.26\"],\"title\":\"added\n'\\\\'\nescapes\",\"number\":199185,\"url\":\"https://github.com/elastic/kibana/pull/199185\",\"mergeCommit\":{\"message\":\"added\n'\\\\'
escapes (#199185)\\n\\n## Summary\\r\\n\\r\\nThis PR adds
backslash\nescaping to previous code
scan\nfixes.\",\"sha\":\"fb7114222b5bde5846fdda85a2ae0e94797600e2\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[\"8.15\",\"7.17\"],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v9.0.0\",\"branchLabelMappingKey\":\"^v9.0.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"https://github.com/elastic/kibana/pull/199185\",\"number\":199185,\"mergeCommit\":{\"message\":\"added\n'\\\\'
escapes (#199185)\\n\\n## Summary\\r\\n\\r\\nThis PR adds
backslash\nescaping to previous code
scan\nfixes.\",\"sha\":\"fb7114222b5bde5846fdda85a2ae0e94797600e2\"}},{\"branch\":\"8.15\",\"label\":\"v8.15.4\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"},{\"branch\":\"7.17\",\"label\":\"v7.17.26\",\"branchLabelMappingKey\":\"^v(\\\\d+).(\\\\d+).\\\\d+$\",\"isSourceBranch\":false,\"state\":\"NOT_CREATED\"}]}]\nBACKPORT-->\n\nCo-authored-by:
Bryce Buchanan <[email protected]>"}}]}]
BACKPORT-->

Co-authored-by: Larry Gregory <[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 ci:project-deploy-observability Create an Observability project release_note:skip Skip the PR/issue when compiling release notes Team:obs-ux-infra_services Observability Infrastructure & Services User Experience Team v7.17.26 v8.15.4 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants