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.

Sign up for GitHub

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

Jump to bottom

Jwt only #159

Open
wants to merge 32 commits into
base: main
Choose a base branch
from
Open

Jwt only #159

wants to merge 32 commits into from

Conversation

ShaneMander
Copy link
Collaborator

JWT only

@ShaneMander
remove oauth
9bad674
@ShaneMander
Merge branch 'main' into jwt
30bf7ef
@ShaneMander
Merge branch 'main' into jwt
e377956
@ShaneMander
jwt skeleton
00b698f
@ShaneMander
jwt skeleton
4b001bb
@ShaneMander
jwt base service
34ca7d8
@ShaneMander
Merge branch 'main' into jwt
e19ad6f
@ShaneMander
jwt filter
7353b4b
@ShaneMander
jwt filled
ede1c96
@ShaneMander
add password to user
8fd261f
@ShaneMander
email + pass base
7fdf988
@ShaneMander
remove unneeded methods + endpoint
8e8bba6
@ShaneMander
remove unneeded imports
0ee3b6d
@ShaneMander
remove test google endpoints
4adff93
@ShaneMander
fix bugs
e5ced37
@ShaneMander
working jwt + email & pass
9b4c3d6
@ShaneMander
logging + bug fix
b7fda3c
@ShaneMander
Merge branch 'main' of https://github.com/Daggerpov/Spawn-App-Back-End …
333f738 
…into jwt

# Conflicts:
#	src/main/java/com/danielagapov/spawn/Repositories/IUserRepository.java
#	src/main/java/com/danielagapov/spawn/Services/OAuth/OAuthService.java
#	src/main/java/com/danielagapov/spawn/Services/User/IUserService.java
#	src/main/java/com/danielagapov/spawn/Services/User/UserService.java
@ShaneMander
fix import
925cd69
@ShaneMander
correct logger import
a410f54
@ShaneMander
add user constructor without password
fde8010
@ShaneMander
refactor with email + exception handling
01e125f
@ShaneMander
jwt for oauth
5cf9b50
@ShaneMander
document and comment
e404331
@ShaneMander
refresh token
7d00e37
@ShaneMander
daniel's terrible script phase 1 removed new files
225180a
@ShaneMander
daniel's terrible script phase 2 reverted User
10cd4ea
@ShaneMander
re-add refresh-token
6e24ff2
@Daggerpov Daggerpov marked this pull request as draft February 11, 2025 22:17
Daggerpov
Daggerpov requested changes Feb 11, 2025
View reviewed changes
Copy link
Owner

@Daggerpov Daggerpov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting for build checks to pass, since we don't want to merge this in and have it depend on the other changes in #132 . Ideally, we split them to where this one can be merged alone and still work

@Daggerpov Daggerpov mentioned this pull request Feb 11, 2025
Open
@ShaneMander
correct failing build + change oauthcontroller to authcontroller + ge…
88ce1e3 
…nerate refresh token
@ShaneMander
Merge branch 'main' of https://github.com/Daggerpov/Spawn-App-Back-End …
f2610e9 
…into jwt-only

# Conflicts:
#	src/main/java/com/danielagapov/spawn/Services/User/UserService.java
@ShaneMander
correct bad import
99ae62f
@ShaneMander ShaneMander marked this pull request as ready for review February 12, 2025 05:50
@ShaneMander
Copy link
Collaborator Author

@Daggerpov Breaking changes: the oauth endpoint no longer exists. It has been renamed to auth so email + pass and oauth sign-in can be accessed from the same controller

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Daggerpov Daggerpov Awaiting requested review from Daggerpov

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ShaneMander @Daggerpov