Skip to content

Commit

Permalink
[8.x] [HTTP] Set explicit access for `public` HTTP APIs 2 (e…
Browse files Browse the repository at this point in the history
…lastic#192579) (elastic#193938)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[HTTP] Set explicit access for `public` HTTP APIs 2
(elastic#192579)](elastic#192579)

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

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

<!--BACKPORT [{"author":{"name":"Jean-Louis
Leysens","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-25T07:25:12Z","message":"[HTTP]
Set explicit access for `public` HTTP APIs 2 (elastic#192579)\n\n##
Summary\r\n\r\nPart 2 of this
[PR](elastic#192554). See\r\nmore info
there.\r\n\r\n<img width=\"229\" alt=\"Screenshot 2024-09-11 at 14 50
11\"\r\nsrc=\"https://github.com/user-attachments/assets/585f1ce8-2987-4355-a4b1-e13705646b48\">\r\n\r\n---------\r\n\r\nCo-authored-by:
shahzad31
<[email protected]>","sha":"244ff7b2c37f66518dbb0d7121a8d02b1405fe93","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Feature:http","Team:Core","release_note:skip","v9.0.0","ci:project-deploy-observability","Team:obs-ux-management","v8.16.0","backport:version"],"title":"[HTTP]
Set explicit access for `public` HTTP APIs
2","number":192579,"url":"https://github.com/elastic/kibana/pull/192579","mergeCommit":{"message":"[HTTP]
Set explicit access for `public` HTTP APIs 2 (elastic#192579)\n\n##
Summary\r\n\r\nPart 2 of this
[PR](elastic#192554). See\r\nmore info
there.\r\n\r\n<img width=\"229\" alt=\"Screenshot 2024-09-11 at 14 50
11\"\r\nsrc=\"https://github.com/user-attachments/assets/585f1ce8-2987-4355-a4b1-e13705646b48\">\r\n\r\n---------\r\n\r\nCo-authored-by:
shahzad31
<[email protected]>","sha":"244ff7b2c37f66518dbb0d7121a8d02b1405fe93"}},"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/192579","number":192579,"mergeCommit":{"message":"[HTTP]
Set explicit access for `public` HTTP APIs 2 (elastic#192579)\n\n##
Summary\r\n\r\nPart 2 of this
[PR](elastic#192554). See\r\nmore info
there.\r\n\r\n<img width=\"229\" alt=\"Screenshot 2024-09-11 at 14 50
11\"\r\nsrc=\"https://github.com/user-attachments/assets/585f1ce8-2987-4355-a4b1-e13705646b48\">\r\n\r\n---------\r\n\r\nCo-authored-by:
shahzad31
<[email protected]>","sha":"244ff7b2c37f66518dbb0d7121a8d02b1405fe93"}},{"branch":"8.x","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jean-Louis Leysens <[email protected]>
  • Loading branch information
kibanamachine and jloleysens authored Sep 25, 2024
1 parent 072fce5 commit 41bb84d
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 3 deletions.
1 change: 1 addition & 0 deletions x-pack/plugins/logstash/server/routes/pipelines/delete.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ export function registerPipelinesDeleteRoute(router: LogstashPluginRouter) {
{
path: '/api/logstash/pipelines/delete',
options: {
access: 'public',
summary: `Delete managed Logstash pipelines`,
},
validate: {
Expand Down
1 change: 1 addition & 0 deletions x-pack/plugins/logstash/server/routes/pipelines/list.ts
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ export function registerPipelinesListRoute(router: LogstashPluginRouter) {
{
path: '/api/logstash/pipelines',
options: {
access: 'public',
summary: `Get all managed Logstash pipelines`,
},
validate: false,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,6 @@ export const syntheticsAppRestApiRoutes: SyntheticsRestApiRouteFactory[] = [
disableSyntheticsRoute,
getServiceLocationsRoute,
getSyntheticsProjectMonitorsRoute,
getAllSyntheticsMonitorRoute,
installIndexTemplatesRoute,
runOnceSyntheticsMonitorRoute,
testNowMonitorRoute,
Expand Down Expand Up @@ -109,6 +108,7 @@ export const syntheticsAppPublicRestApiRoutes: SyntheticsRestApiRouteFactory[] =
addPrivateLocationRoute,
deletePrivateLocationRoute,
getPrivateLocationsRoute,
getAllSyntheticsMonitorRoute,
getSyntheticsMonitorRoute,
addSyntheticsMonitorRoute,
editSyntheticsMonitorRoute,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,11 @@ import { syntheticsMonitorType } from '../../../common/types/saved_objects';
export const getAllSyntheticsMonitorRoute: SyntheticsRestApiRouteFactory = () => ({
method: 'GET',
path: SYNTHETICS_API_URLS.SYNTHETICS_MONITORS,
validate: {
query: QuerySchema,
validate: {},
validation: {
request: {
query: QuerySchema,
},
},
handler: async (routeContext): Promise<any> => {
const { request, savedObjectsClient, syntheticsMonitorClient } = routeContext;
Expand Down

0 comments on commit 41bb84d

Please sign in to comment.