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

restifyjs setup without auth middleware #598

Merged
merged 2 commits into from
Dec 30, 2023
Merged

restifyjs setup without auth middleware #598

merged 2 commits into from
Dec 30, 2023

Conversation

binaryk
Copy link
Collaborator

@binaryk binaryk commented Dec 30, 2023

Fixed

  • restifyjs setup without auth middleware

Copy link

what-the-diff bot commented Dec 30, 2023

PR Summary

  • Update to the Routes Definition

    This pull request includes a change to the file that outlines the path that data takes through our application, known as RoutesDefinition.php. The specific alteration involves the removal of a line of code related to user authentication (confirming users are who they say they are). Previously, we used a piece of software referred to as RestifySanctumAuthenticate for this task.

  • Adjustments to the restifyjs.setup Route

    In conjunction with the above change, this pull request modifies a route named restifyjs.setup and removes the RestifySanctumAuthenticate middleware, replacing it with 'auth:sanctum'. Middleware can be thought of as a gatekeeper for data, helping manage and process requests from users. Specifically, 'auth:sanctum' middleware is now the new user authentication tool in use.

The net effect of this change is to substitute the tool we use to authenticate users. This could potentially provide more effective handling of user information and enhance security.

@binaryk
Copy link
Collaborator Author

binaryk commented Dec 30, 2023

Fixing #593

@binaryk binaryk merged commit 1d41e72 into 8.x Dec 30, 2023
@binaryk binaryk deleted the fix/restifyjs branch December 30, 2023 12:55
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.

1 participant