airbyte
airbyte copied to clipboard
✨Source Okta: migrate to low-code
Migrating Source Okta to YamlDeclarativeSource (Low-code) Airbyte CDK
Resolves 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 <Token> header, which is not supported by the default Oauth2 and Bearer Authenticators (they add the Authorization: Bearer <Token> header) so I made custom components for the same.
The latest updates on your projects. Learn more about Vercel for Git ↗︎
1 Ignored Deployment
| Name | Status | Preview | Comments | Updated (UTC) |
|---|---|---|---|---|
| airbyte-docs | ⬜️ Ignored (Inspect) | Visit Preview | May 21, 2024 3:10pm |
@tolik0 I've added the unit tests back. Added new unit tests for config migration, fixed the issue with the wrong primary keys, updated airbyte-cdk and expected records. This PR is ready for review.
Also, could you please run airbyte-ci format locally and push those changes here?
I've ran the airbyte-ci format fix all command 4-5 times and even though it runs successfully locally, it is failing here in the pipelines.
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) All the other files are properly formatted so we can ignore that.