#157 Protect /projects and /platforms routes with auth guard#302
Merged
Eniola3321 merged 1 commit intoANYTECHS:mainfrom Apr 29, 2026
Merged
#157 Protect /projects and /platforms routes with auth guard#302Eniola3321 merged 1 commit intoANYTECHS:mainfrom
Eniola3321 merged 1 commit intoANYTECHS:mainfrom
Conversation
- Added /projects and /platforms to protected routes list - Ensured unauthenticated users are redirected to /login - Used prefix matching to cover nested routes - Fixed redirect logic to avoid unnecessary navigation and loops - Removed unused React Router hooks (useLocation, useNavigate) - Maintained access for authenticated users
|
@Tobiloba0 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! 🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR fixes an access control issue where the /projects and /platforms pages were not protected by the authentication guard.
Previously, unauthenticated users could access these routes directly via URL. This update ensures they are properly restricted.
✨ Changes
Added /projects and /platforms to the protected routes list in AuthProvider
Updated route matching to use startsWith for better coverage (supports nested routes)
Ensured unauthenticated users are redirected to /login
Preserved access for authenticated users
Improved redirect flow to prevent unnecessary navigation and potential loops
Removed unused React Router hooks (useLocation, useNavigate) to align with Next.js routing
✅ Acceptance Criteria
/projects is protected
/platforms is protected
Unauthenticated users are redirected to /login
Authenticated users can access both pages normally
No redirect loop introduced
Closes #157