Skip to content

Commit b9cb7d0

Browse files
committed
remove id validation, it's not used in logic anyway
1 parent 9d61ffc commit b9cb7d0

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

x-pack/plugins/logstash/public/application/components/pipeline_editor/pipeline_editor.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -125,11 +125,11 @@ class PipelineEditorUi extends React.Component {
125125

126126
onPipelineSave = () => {
127127
const { pipelineService, toastNotifications, intl } = this.props;
128-
const { id } = this.state.pipeline;
128+
const { id, ...pipelineToStore } = this.state.pipeline;
129129
return pipelineService
130130
.savePipeline({
131131
id,
132-
upstreamJSON: this.state.pipeline,
132+
upstreamJSON: pipelineToStore,
133133
})
134134
.then(() => {
135135
toastNotifications.addSuccess(

x-pack/plugins/logstash/server/routes/pipeline/save.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ export function registerPipelineSaveRoute(router: IRouter, security?: SecurityPl
2222
id: schema.string(),
2323
}),
2424
body: schema.object({
25-
id: schema.maybe(schema.string()), // id might be provided when cloning pipeline
2625
description: schema.maybe(schema.string()),
2726
pipeline: schema.string(),
2827
settings: schema.maybe(schema.object({}, { unknowns: 'allow' })),

0 commit comments

Comments
 (0)