-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🎉 Source Salesforce: Support oauth'ing into sandbox environment #7877
Conversation
…option of the Salesforce connector
/test connector=source-salesforce
|
/test connector=source-salesforce-sandbox
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vitaliizazmic is it possible instead of completely replicating the connector, to only inherit from the salesforce docker image, similar to source-file-secure
?
@@ -496,6 +496,13 @@ | |||
documentationUrl: https://docs.airbyte.io/integrations/sources/salesforce | |||
icon: salesforce.svg | |||
sourceType: api | |||
- name: SalesforceSandbox |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- name: SalesforceSandbox | |
- name: Salesforce (Sandbox) |
What
When oauth'ing a salesforce user into a sandbox environment, we need to send them to
test.salesforce.com
rather thanlogin.salesforce.com
Our current server implementation of salesforce oauth always assumes they are not trying to use a sandbox environment and redirects them tologin.salesforce.com
.Closes #7722
How
Temporary solution was created according to proposed algorithm into issue #7722
test.salesforce.com
.Pre-merge Checklist
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog example