airbyte icon indicating copy to clipboard operation
airbyte copied to clipboard

✨Source Okta: migrate to low-code

Open pabloescoder opened this issue 1 year ago • 3 comments

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.

pabloescoder avatar Mar 26 '24 21:03 pabloescoder

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

vercel[bot] avatar Mar 26 '24 21:03 vercel[bot]

@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.

pabloescoder avatar May 15 '24 21:05 pabloescoder

/format-fix

Format-fix job started... Check job output.

🟦 Job completed successfully (no changes).

lazebnyi avatar May 16 '24 13:05 lazebnyi

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.

pabloescoder avatar May 20 '24 21:05 pabloescoder