Syndesis: Drop integration id and operation id from the flow id

Created on 27 Jun 2019  路  4Comments  路  Source: syndesisio/syndesis

This is a...


[ ] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report  
[ ] Documentation issue or request

Description

Currently we're creating flow id of the API provider generated flows by concatenating integration id, :flows: and operation id (of the OpenAPI operation).

We should store the operation id in the flow metadata, and modify the logic to match the operation with the flow based on the metadata.

The existing flows need to be migrated such that flow ids are regenerated and the operation id from the flow id is placed into the metadata of the flow.

cabug grouserver ignorqe prip0

Most helpful comment

Yeah, let me slap a ignore/qe on this and close.

All 4 comments

This also matches the pattern we've adopted for conditional flows, where a flow's metadata is used to provide the linking between flows. It also ensures the ID format stays opaque and neutral and can therefore be fairly safely generated by the backend or frontend, which is kinda crucial given that either side is responsible for creating objects in an integration during various use cases.

Relates to #5697

@zregvart is there anything you need to get validated on this issue? Seems to me like an internal thing that QE doesn't realistically get chance to validate properly

Yeah, let me slap a ignore/qe on this and close.

Was this page helpful?
0 / 5 - 0 ratings