Skip to content
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

Update instance wide oauth parameters #6309

Merged
merged 2 commits into from
Sep 20, 2021
Merged

Conversation

ChristopheDuong
Copy link
Contributor

@ChristopheDuong ChristopheDuong commented Sep 20, 2021

What

Make sure only one global instance wide OAuth parameters is set per connector definition id

How

Checks if a oauth param already exists for a connector definition id before creating a new one.
Updates the previous params if already exists

@ChristopheDuong ChristopheDuong temporarily deployed to more-secrets September 20, 2021 11:09 Inactive
@github-actions github-actions bot added the area/platform issues related to the platform label Sep 20, 2021
@ChristopheDuong ChristopheDuong temporarily deployed to more-secrets September 20, 2021 12:02 Inactive
@ChristopheDuong ChristopheDuong temporarily deployed to more-secrets September 20, 2021 16:39 Inactive
@ChristopheDuong ChristopheDuong temporarily deployed to more-secrets September 20, 2021 17:04 Inactive
@ChristopheDuong ChristopheDuong merged commit 4f1da3e into master Sep 20, 2021
@ChristopheDuong ChristopheDuong deleted the chris/oauth-params branch September 20, 2021 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants