search

syndesisio / syndesis-verifier

Syndesis Verifier Service
Apache License 2.0
2 stars 8 forks source link

Can't Verify Saleforce Connection #12

Closed chirino closed 6 years ago

chirino commented 7 years ago

See:

syndesis
rhuss commented 7 years ago

Think we already had that before. will check it later again.

chirino commented 7 years ago

@zregvart @rhuss any updates?

zregvart commented 7 years ago

@chirino @rhuss I've created a PR, seems to be OK for me (tested with curl), but I think that this uncovers a deeper issue: the connector name is not passed on to the verifier (the verifier uses a fixed connector name), and when considering that connectors can provide some of the options (and probably most will) this might be an issue going forward. Not sure if it's a UI, REST or conceptual issue, but I'm leaning towards the latter.

rhuss commented 7 years ago

The mismatch is laid out in https://github.com/syndesisio/syndesis-rest/issues/219, too. The problem is that when we create a Syndesis Connection we have no action yet and hence cannot select a specific Camel Connector. Actually the idea is that we are taking an opinionated approach and choose one of the connectors as the "verifying connector" with the assumption that all connectors in the same 'family' can be verified the same way.

zregvart commented 6 years ago

This works nicely now.