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

feat(integration): identify supported OAuth integrations through global secrets #791

Conversation

jvallesm
Copy link
Collaborator

Because

  • We want to let CE users set up OAuth configuration for components.
  • Until now, OAuth was considered as supported based on the component
    definition.

This commit

  • Consider OAuth supported when both the component definition and the
    component global secrets have a complete OAuth configuration.

@jvallesm jvallesm self-assigned this Oct 30, 2024
Copy link

linear bot commented Oct 30, 2024

@jvallesm jvallesm force-pushed the jvalles/ins-6570-pass-oauth-client-and-secret-ids-to-backend-and-frontend branch 5 times, most recently from c37edff to a53b09a Compare October 30, 2024 15:44
@jvallesm jvallesm marked this pull request as ready for review October 30, 2024 15:44
donch1989
donch1989 previously approved these changes Oct 31, 2024
@jvallesm jvallesm force-pushed the jvalles/ins-6570-pass-oauth-client-and-secret-ids-to-backend-and-frontend branch from a53b09a to f39a3ad Compare October 31, 2024 07:06
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ℹ️ @chuang8511 I extracted the OAuth support logic from the individual components as, at least for now, it will be the same code. Have a look at how I added it in the individual components github and slack (both in the component's main and on the store initialiser).

@jvallesm jvallesm merged commit 5a96453 into main Oct 31, 2024
11 checks passed
@jvallesm jvallesm deleted the jvalles/ins-6570-pass-oauth-client-and-secret-ids-to-backend-and-frontend branch October 31, 2024 07:39
jvallesm added a commit to instill-ai/api-gateway that referenced this pull request Oct 31, 2024
…#264)

Because

- instill-ai/pipeline-backend#791 introduced a
environment variable dependency in the integration tests.

This commit

- Adds the environment variable to `instill-cloud` setup in CI.
donch1989 pushed a commit that referenced this pull request Nov 5, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.46.0-beta](v0.45.2-beta...v0.46.0-beta)
(2024-11-05)


### Features

* add `condition` field support for iterator
([#803](#803))
([04b1252](04b1252))
* add markdowns per pages
([#792](#792))
([3ee428e](3ee428e))
* add resolution field
([#808](#808))
([f15f6bf](f15f6bf))
* add task sync
([#793](#793))
([41a1eeb](41a1eeb))
* **component,audio:** add TASK_DETECT_ACTIVITY and TASK_SEGMENT
([#762](#762))
([9e92a31](9e92a31))
* **component,http:** refactor `restapi` component to `http` component
([#797](#797))
([c2b1862](c2b1862))
* **component:** add error handling for missing conversation
([#806](#806))
([54cc616](54cc616))
* **component:** inject global secrets as environment variables
([#786](#786))
([8d842a6](8d842a6))
* convert time type to string
([#809](#809))
([7de8465](7de8465))
* **googledrive:** add the tasks for google drive
([#725](#725))
([b6fe968](b6fe968))
* **integration:** identify supported OAuth integrations through global
secrets
([#791](#791))
([5a96453](5a96453))
* **minio:** import updated minio package and add tag on file upload
([#779](#779))
([ef86318](ef86318))
* revamp Instill Format
([#774](#774))
([24153e2](24153e2))
* support `length` attribute for array data
([#810](#810))
([fb4f4f7](fb4f4f7))
* **web:** refactor the web operator
([#772](#772))
([ae4e3c2](ae4e3c2))


### Bug Fixes

* **component,image:** fix missing show score draw
([#801](#801))
([a405bf7](a405bf7))
* fix bug not to return error if there is no app or conversation
([#816](#816))
([a946cfd](a946cfd))
* fix iterator upstream check
([#794](#794))
([671971f](671971f))
* **run:** add metadata retention handler
([#800](#800))
([25ec0c2](25ec0c2))
* **run:** add namespace id in response
([#811](#811))
([8d29ffb](8d29ffb))
* **run:** rename pipeline run columns and fix tests
([#776](#776))
([98f1e00](98f1e00))
* **slack:** correct link to OAuth config in documentation
([#805](#805))
([aa0752d](aa0752d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants