-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 Okta: migrate to low-code #36509
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
airbyte-integrations/connectors/source-okta/source_okta/config_migration.py
Show resolved
Hide resolved
airbyte-integrations/connectors/source-okta/source_okta/custom_authenticators.py
Show resolved
Hide resolved
airbyte-integrations/connectors/source-okta/source_okta/manifest.yaml
Outdated
Show resolved
Hide resolved
…irbytehq/airbyte into source-okta-migrate-to-lowcode
@tolik0 Also, could you please run airbyte-ci format locally and push those changes here? |
/format-fix
|
The check for formatting errors pipeline is failing only on the unit_tests folder (which is excluded from the airbyte-ci path for formatting - see here) |
Migrating Source Okta to
YamlDeclarativeSource (Low-code)
Airbyte CDKResolves https://github.com/airbytehq/airbyte-internal-issues/issues/6757
No breaking changes, I've tried to replicate everything that was there in the previous python based source. Since the source supported 2 methods of authentication, I have used a SelectiveAuthenticator. Both authentication methods needed the Authorization: SSWS header, which is not supported by the default Oauth2 and Bearer Authenticators (they add the Authorization: Bearer header) so I made custom components for the same.