syndesisio / syndesis-rest

The API for Syndesis - a flexible, customizable, cloud-hosted platform that provides core integration capabilities as a service. It leverages Red Hat's existing product architecture using OpenShift Online/Dedicated and Fuse Integration Services.
https://syndesis-staging.b6ff.rh-idev.openshiftapps.com/api/v1/
Apache License 2.0
6 stars 17 forks source link

Validating Salesforce connection does not seem right. #339

Closed chirino closed 7 years ago

chirino commented 7 years ago

Got a validation error on an OK response.

red_hat_ipaas
zregvart commented 7 years ago

According to AbstractClientBase.java#L150 Jetty threw an exception. Can you set Jetty logger to debug so it's visible? Or is there a SalesforceException logged somewhere?

rhuss commented 7 years ago

Let me check that. There has been some slight changes in the upstream Verifier API which I need to verify. will work on this today.

lburgazzoli commented 7 years ago

@rhuss if you need any help with camel ping me

zregvart commented 7 years ago

I can't reproduce this, I was struggling to find the exact screen (the SObject Id Name was removed in #338), but this scenario (username-password) works OK with my Salesforce organization and user.

Salesforce is quite picky about letting anyone in, few things of note:

rhuss commented 7 years ago

Just verified that it works for me, too.

However, I find the UI a bit confusing: I would expect User Name and Password to be close together, with User Name before password. Also, the hints within the text input fields are too long. I think a tooltip would be more appropriate here. The ice of the cake would be if the UI logic could ensure that either refreshToken or (user, password) can be entered by disabling fields when one is added. But I guess that's too much (because this info is probably not available at the point)

image

rhuss commented 7 years ago

Ok, I didn't tried it with username / password (and have probably still an old version without SObject ID).

chirino commented 7 years ago

Actually SObjectID should not have been there. you have the latest version.

zregvart commented 7 years ago

As long as we're throwing screenshots, here is mine

selection_303

chirino commented 7 years ago

I've not been able to reproduce the error. Closing out.