Skip to content
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

Ntrnl 450 add authentication middleware #29

Merged
merged 5 commits into from
Jul 11, 2024

Conversation

harley-harris
Copy link
Contributor

@harley-harris harley-harris commented Jul 10, 2024

This PR supercedes #21

JIRA link

NTRNL-450

Description

  • Add authentication middleware

To test this PR, authentication middleware has been mounted on info route. Use a valid COLA JWT to test auth in production env.

Work checklist

  • Tests added where applicable
  • No vulnerability added
Screenshot 2024-07-10 at 11 57 54

@harley-harris harley-harris requested a review from a team July 10, 2024 12:29
@harley-harris harley-harris self-assigned this Jul 10, 2024
Copy link

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 100% 246/246
🟢 Branches
95.45% (+0.22% 🔼)
21/22
🟢 Functions 100% 25/25
🟢 Lines 100% 224/224
Show new covered files 🐣
St.
File Statements Branches Functions Lines
🟢
... / authentication.middleware.ts
100% 100% 100% 100%

Test suite run success

47 tests passing in 23 suites.

Report generated by 🧪jest coverage report action from 6d9d13e

@harley-harris harley-harris mentioned this pull request Jul 10, 2024
2 tasks
@harley-harris harley-harris merged commit 3fdb6c2 into main Jul 11, 2024
7 checks passed
@harley-harris harley-harris deleted the NTRNL-450-add-authentication-middleware branch July 11, 2024 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants