[FEATURE] Optional auth for AutosysSensor

maxim-mityutko commented 9 months ago

Closes #86

Description

Making authentication optional for AutosysSensor. If airflow_cluster_auth is provided, it will be used to obtain the authentication token. The token then will be included in the request headers. Otherwise the call will happen without auth.

Motivation and Context

Skip authentication token acquisition if it's not required, reduces the amount of calls to the credentials store and / or authentication service, eg. Okta. Streamlines and simplifies user experience.

How Has This Been Tested?

Built wheel with the change, used wheel as a cluster library instead of the original brickflow distribution from PyPi. Executed workflow with AutosysSensor both with and without authentication. Both scenarios succeeded.

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (548b6cb) 90.27% compared to head (9fc045f) 90.32%. Report is 3 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #87 +/- ## ========================================== + Coverage 90.27% 90.32% +0.05% ========================================== Files 22 22 Lines 3361 3381 +20 ========================================== + Hits 3034 3054 +20 Misses 327 327 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

maxim-mityutko commented 8 months ago

@asingamaneni @stikkireddy can you please review this PR or let me if anything else should be changed / added?

Nike-Inc / brickflow