Skip to content

feat: ACNA-3659 | Add auth_handler to OW#206

Merged
shazron merged 4 commits intomasterfrom
ACNA-3659
Mar 20, 2025
Merged

feat: ACNA-3659 | Add auth_handler to OW#206
shazron merged 4 commits intomasterfrom
ACNA-3659

Conversation

@amulyakashyap09
Copy link
Copy Markdown
Contributor

@amulyakashyap09 amulyakashyap09 commented Mar 10, 2025 •
edited
Loading

Description

  • Added auth_handler to openwhisk for the bearer token authorisation
  • Added condition check for auth_handler

Related Issue

ACNA-3659
ACNA-3602

Motivation and Context

How Has This Been Tested?

  • Unit test [🟢]
  • Manual API Testing Completed [🟢]

Screenshots (if appropriate):

Screenshot 2025-03-11 at 10 49 12

Types of changes

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

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 10, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (8d531a9) to head (6ccd4e7).
Report is 1 commits behind head on master.

Additional details and impacted files 
@@            Coverage Diff            @@
##            master      #206   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines         1339      1346    +7     
  Branches       346       349    +3     
=========================================
+ Hits          1339      1346    +7

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@amulyakashyap09 amulyakashyap09 mentioned this pull request Mar 10, 2025
Merged
10 tasks
@amulyakashyap09 amulyakashyap09 changed the title bugfix: ACNA-3659 | Fixing-bugs feat: ACNA-3659 | Add auth_handler to OW Mar 11, 2025
shazron
shazron requested changes Mar 11, 2025
View reviewed changes
Comment thread src/deploy-actions.js Outdated
Comment thread src/deploy-actions.js Outdated
Comment thread src/print-action-logs.js
Comment thread src/print-action-logs.js Outdated
Comment thread src/undeploy-actions.js Outdated
Comment thread src/undeploy-actions.js Outdated
@amulyakashyap09 amulyakashyap09 requested a review from shazron March 11, 2025 06:44
shazron
shazron requested changes Mar 11, 2025
View reviewed changes
Comment thread test/deploy.actions.test.js Outdated
@amulyakashyap09 amulyakashyap09 requested a review from shazron March 11, 2025 07:16
shazron
shazron reviewed Mar 19, 2025
View reviewed changes
Comment thread src/deploy-actions.js
@shazron shazron mentioned this pull request Mar 19, 2025
Merged
10 tasks
@shazron shazron merged commit a9b9e5c into master Mar 20, 2025
10 checks passed
@shazron shazron deleted the ACNA-3659 branch March 20, 2025 05:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shazron shazron shazron approved these changes

@arjuncooliitr arjuncooliitr arjuncooliitr approved these changes

@sandeep-paliwal sandeep-paliwal sandeep-paliwal approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@amulyakashyap09 @shazron @arjuncooliitr @sandeep-paliwal