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

Bearer tokens #74

Merged
merged 32 commits into from
Mar 17, 2024
Merged

Bearer tokens #74

merged 32 commits into from
Mar 17, 2024

Conversation

ryanrdoherty
Copy link
Member

@ryanrdoherty ryanrdoherty commented Nov 6, 2023 •
edited
Loading

Current:

  • Session = jsessionid
  • Logged in = wdk_check_auth cookie + User on backend session
  • Guest = no cookie + User on backend session

Goal:

  • Session = none
  • Logged in = Authorization header/cookie for non-guest
  • Guest = Authorization header/cookie for guest

@ryanrdoherty ryanrdoherty self-assigned this Nov 6, 2023
ryanrdoherty and others added 23 commits December 6, 2023 01:45
@ryanrdoherty
Complete separation of user prefs from user
ddd0f25
@ryanrdoherty
Remove incorrect documentation
42625c2
@ryanrdoherty
Add signature back into OAuth API (not deprectated after all)
72be5ba
@ryanrdoherty
Merge branch 'master' into bearer-tokens
01a27b9
@ryanrdoherty
Make names more consistent
cf0c5d3
@ryanrdoherty
Changes to support bearer tokens and remove accountdb access from WDK
ffff175
@ryanrdoherty
Merge branch 'master' into bearer-tokens
ac484d9
@ryanrdoherty
checkpoint commit
23c59fc
@ryanrdoherty
Fix up user-related services and clean out code a bit
95bc896
@ryanrdoherty
Comply with OAuth upgrade in install
fa8c778
@ryanrdoherty
Merge WdkOAuthClientWrapper into UserFactory, but split off managemen…
54d7c3c 
…t of the DB users table and password emails into their own classes
@ryanrdoherty
Convert session to temporary user data store
1ad6449
@ryanrdoherty
Stop using SessionProxy interface
cc2fe2c
@ryanrdoherty
Switch to ConcurrentHashMap in case user has multiple requests going …
6c7777a 
…at the same time
@ryanrdoherty
Merge pull request #81 from VEuPathDB/temporary-data
72c89a8 
Convert session to temporary user data store
@ryanrdoherty
ditch accountdb
5b5cd90
@ryanrdoherty
Merge branch 'master' into bearer-tokens
ba94756
@ryanrdoherty
Remove accountDb from RNG
3105dce
@ryanrdoherty
Produce user factories on demand vs keeping one around
16bca8d
@ryanrdoherty
Remove acctdb references
6882f15
@ryanrdoherty
why is there an NPE?
af09d25
@ryanrdoherty
Fix NPE
5afa814
@ryanrdoherty
Comply with new oauth API and try to return better errors
facb734
ryanrdoherty
ryanrdoherty commented Mar 17, 2024
View reviewed changes
Model/src/main/java/org/gusdb/wdk/cache/TemporaryUserDataStore.java Outdated
}

public void invalidate() {
clear();
Copy link
Member Author

Choose a reason for hiding this comment

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

Thinking now this is probably not a good idea because it could break other requests. What we want is for the map to be removed from the store but maybe not clear the map. Eventually the whole thing will be garbage collected.

@ryanrdoherty ryanrdoherty marked this pull request as ready for review March 17, 2024 11:41
@ryanrdoherty ryanrdoherty merged commit 7f04a63 into master Mar 17, 2024
1 check passed
@ryanrdoherty ryanrdoherty deleted the bearer-tokens branch March 17, 2024 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ryanrdoherty ryanrdoherty

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ryanrdoherty