Skip to content

[fix]: add token validation before fetching user data - #124

Open
AnkitRewar11 wants to merge 1 commit into
layer5io:masterfrom
AnkitRewar11:fix/recognition-login-fix
Open

AnkitRewar11 wants to merge 1 commit into
layer5io:masterfrom
AnkitRewar11:fix/recognition-login-fix

Conversation

@AnkitRewar11

@AnkitRewar11 AnkitRewar11 commented Sep 21, 2026

Copy link
Copy Markdown

Notes for Reviewers

This PR fixes #121

Fixed the login state in the navigation.

  • Only fetches the profile when a valid token is available.
  • Restores the user state after a successful profile request.
  • Hides the Login button for logged-in users.
  • Prevents Authorization: Bearer null requests.

Note: CORS still needs to be configured on the Cloud API side.

Signed commits

  • Yes, I signed my commits.

Signed-off-by: ankitrewar11 <rewarankit18@gmail.com>
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 44 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 543931d7-11fb-4ceb-9fb4-9f29617a12f1

📥 Commits

Reviewing files that changed from the base of the PR and between 0f048b8 and 3662d56.

📒 Files selected for processing (1)
  • src/sitecomponents/Navigation/index.js

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added area/ci Continuous integration | Build and release component/kanvas-docs labels Sep 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ci Continuous integration | Build and release component/kanvas-docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Auth token missing in profile API request

1 participant