Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Currently we would be attach OAuth framework specific things when we identify(
auth.type
) from the destination definition config to beOAuth
, which would mean user would be restricted to go ahead with onlyOAuth
not anything else.In the past there have been use-cases where a destination would need to support multiple authentication mechanisms. We would need to provide multiple authentication mechanisms in the UI
In order to fulfil this, we have added
auth.type
asoptionalAuth
in the destination definition config. We are using this to identify if a destination optionally selectedOAuth
& also confirm if account id information is available in destination.config for attaching OAuth framework specific thingsLinear Ticket
Resolves INT-2832
Security