projects/file-explorer
projects/file-explorer
1
0
Fork 1
Code Issues 1 Pull Requests 2 Actions Packages Projects Releases Wiki Activity

implement authentication with oauth #8

Open
csimonis wants to merge 4 commits from feature/oauth into main
pull from: feature/oauth
merge into: projects:main
Conversation 1 Commits 4 Files Changed 11 +1822 -106
csimonis commented 2024-12-11 20:39:31 +00:00
Owner
Copy Link
No description provided.
csimonis added 1 commit 2024-12-11 20:39:31 +00:00
csimonis force-pushed feature/oauth from 3a841be03e to 07c36a5737 2024-12-11 21:01:32 +00:00 Compare
csimonis added 1 commit 2024-12-11 21:19:43 +00:00
csimonis added 1 commit 2024-12-11 21:20:52 +00:00
csimonis force-pushed feature/oauth from 36009ef834 to e0828cff8d 2024-12-12 16:01:43 +00:00 Compare
csimonis added 1 commit 2024-12-12 16:02:20 +00:00
csimonis added 1 commit 2024-12-12 17:30:11 +00:00
csimonis added a new dependency 2024-12-18 11:30:47 +00:00
angular/ems-frontend#6 - style employee list
csimonis removed a dependency 2024-12-18 11:31:01 +00:00
angular/ems-frontend#6 - style employee list
csimonis force-pushed feature/oauth from a63306becb to ae33db2a98 2024-12-21 13:18:51 +00:00 Compare
jank1619 requested changes 2024-12-21 13:22:32 +00:00
jank1619 left a comment
Contributor
Copy Link

I reviewed ur changes. Might work on file previewing now maybe.

I reviewed ur changes. Might work on file previewing now maybe.
src/Security/Authenticator.php Outdated
@ -0,0 +38,4 @@
public function onAuthenticationFailure(Request $request, AuthenticationException $exception): ?Response
{
dd($request );
jank1619 commented 2024-12-21 13:21:33 +00:00
Contributor
Copy Link

This might not be the best idea

This might not be the best idea
csimonis marked this conversation as resolved
src/Security/UserProvider.php Outdated
@ -0,0 +55,4 @@
protected function createResourceOwner(array $response, AccessToken $token)
{
dd($response);
jank1619 commented 2024-12-21 13:22:10 +00:00
Contributor
Copy Link

This one isnt that good either

This one isnt that good either
csimonis marked this conversation as resolved
csimonis force-pushed feature/oauth from ae33db2a98 to 77951dd48b 2024-12-21 18:13:42 +00:00 Compare
This pull request doesn't have enough required approvals yet. 0 of 1 official approvals granted.
This branch is out-of-date with the base branch
You are not authorized to merge this pull request.
View command line instructions.

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/oauth:feature/oauth
git checkout feature/oauth
Sign in to join this conversation.
Reviewers
No Reviewers
jank1619
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
csimonis
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: projects/file-explorer#8
No description provided.
Delete Branch "feature/oauth"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?