Skip to content

refactor(auth): consolidate authentication middleware - #298

Merged
Tybravo merged 1 commit into
SwiftChainn:mainfrom
ayo-mide55:fix/issue-185
Oct 4, 2026
Merged

Tybravo merged 1 commit into
SwiftChainn:mainfrom
ayo-mide55:fix/issue-185

Conversation

@ayo-mide55

Copy link
Copy Markdown
Contributor

Delegates the legacy auth.ts and authMiddleware.ts import paths to the canonical database-backed authenticate middleware. This preserves existing imports while consistently rejecting missing, invalid, suspended, and banned accounts; role authorization remains available through the compatibility helper.

Closes #185

@drips-wave

drips-wave Bot commented Sep 30, 2026

Copy link
Copy Markdown

@ayo-mide55 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Tybravo

Tybravo commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

@ayo-mide55

Thanks for your contribution

However I can't merge this PR because I don't know why i am seeing 2 participant when i assigned only one contributor.

Please fix this.

@ayo-mide55

Copy link
Copy Markdown
Contributor Author

The commit attribution and formatting updates are in place. Please approve the pending CI run, then review and merge this PR when the checks pass.

@ayo-mide55

Copy link
Copy Markdown
Contributor Author

@Tybravo I checked the current PR history: it contains one commit authored by @ayo-mide55 and has no co-authored commit. GitHub currently lists @ayo-mide55 and you as the two participants. If you are seeing another contributor account, could you point me to it? If those are the two you meant, please consider merging this PR; the assigned contributor is the sole commit author.

@Tybravo

Tybravo commented Oct 4, 2026

Copy link
Copy Markdown
Collaborator

@ayo-mide55

Well-fixed and now we have only 1 contributor as assigned to have legitimately worked on the issue.

Thanks for your contribution to the project

@Tybravo
Tybravo merged commit 1ed70fe into SwiftChainn:main Oct 4, 2026
1 check failed
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.

[Refactor] Consolidate the three duplicate authentication middlewares into one canonical implementation

2 participants