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

Use Spring WebClient #829

Draft
wants to merge 64 commits into
base: main
Choose a base branch
from
Draft

Use Spring WebClient #829

wants to merge 64 commits into from

Conversation

thomaskrause
Copy link
Member

For long running operations on the same API Client (e.g. an export), it could happen that the token expired during the process. This refactors the CommonUI#getClient method to return a client that updates the JWT token before each request.

thomaskrause and others added 20 commits November 3, 2022 19:38
@thomaskrause
Update to Spring Boot 2.5.14
25c33bc
@thomaskrause
Upgrade to Spring Booot 2.7.5
bffdbed
@thomaskrause
Don't configure kotlin stdlib manually
f342428
@thomaskrause
Configure OpenApi to generate the Spring WebClient (does not compile …
403ed4e 
…yet)

The WebClient is a Spring class that can support automatic refreshing of the token. But using this new API means we have to change all calls to the (possible blocking) WebFlux calls.
@thomaskrause
Fix more of the compilation issues from changing to Spring WebClient
b0d0e37
@thomaskrause
Use WebClientResponseException
811d77c
@thomaskrause
Add block() to several of the more straight forward synchronous calls…
00b4bb6 
… to get a file
@thomaskrause
Replace some of the occurences of "collectList().block()" with "toIte…
b8ad22a 
…rable()"
@thomaskrause
Fix remaining compilation issues regarding the new WebClient
d6861a7
@thomaskrause
Merge branch 'main' into feature/refresh-jwt-token
8aaf12a 
# Conflicts:
#	pom.xml
@thomaskrause
Add a "patched" CorpusApi implementation that allows to get the list …
7e10ed0 
…of strings as mono.

If OpenAPITools/openapi-generator#14125 gets merged, we won't need this workaround anymore
@thomaskrause
Use the patched API where the corpus list is fetched
db60d7a
@thomaskrause
Use the patched API where the file list is fetched
9aee36f
@thomaskrause
Attempt to configure a WebClient bean with authentfication
da4c948
@thomaskrause
Fix compilation issues
1151b20
@thomaskrause
Formatting
b4e1fa0
@thomaskrause
Merge branch 'feature/refresh-jwt-token'
ee9ff55 
# Conflicts:
#	pom.xml
@thomaskrause
Excplicitly set the H2 dialect
651e265
@thomaskrause
Removed double dependency entry for the salt-api
6355049
@thomaskrause
Use webclient in ApiClient
4361714
@thomaskrause thomaskrause marked this pull request as draft May 22, 2023 12:09
@thomaskrause thomaskrause self-assigned this May 22, 2023
thomaskrause and others added 29 commits July 27, 2023 17:01
@thomaskrause
Replace usage of subgraph queries with WebClient
4e5afb7
@thomaskrause
Fetch document_browser.json with WebClient
147529a
@thomaskrause
Specify "bearer" auth mode in desktop mode
22b4da2
@thomaskrause
Use WebClient when getting the annotations for the query builders
f34f140
@thomaskrause
Delete corpora using WebClient
89bb035
@thomaskrause
Use WebClient to find results in exporters
0898a11
@thomaskrause
Remove depricated WebSecurityConfigurerAdapter
dff3e85
@thomaskrause
Use WebClient for frequency query
284fb71
@thomaskrause
Use WebClient when importing a corpus
0d38717
@thomaskrause
Use WebClient in group management
fdfbc83
@thomaskrause
Reduce complexity of the filter creation logic
98bb53c
@thomaskrause
Do not use background thread to query corpus list in background
ad753a2
@thomaskrause
Use long polling as push strategy because there are issues with the s…
c0b8b6b 
…essions and Oauth2
@thomaskrause
Use Flux API to get the corpus configurations in the background
ab35323
@thomaskrause
Use Flux API to get example queries
c4113df
@thomaskrause
Use Flux API for initial result page
fc5245c
@thomaskrause
Remove last usages of ApiClient
538b9d7
@thomaskrause
Use query controller to update find search for paging
abe52e7
@thomaskrause
Relocate some classes so the "beans" package is not needed anymore
0ace0a3
@thomaskrause
Reorganize class structure and remove some interfaces with only one i…
cd277b6 
…mplementation
@thomaskrause
Use Flux API in CorpusBrowserPanel
904e696
@thomaskrause
Get metadata in corpus browser via Flux API
5aa7d02
@thomaskrause
Do not execute adapted Flux APIs calls in background thread
afc60b8
@thomaskrause
Do not attempt to get edge annotations for components with an empty name
dda4549
@thomaskrause
Avoid detached UI in CorpusListPanel
7704b34
@thomaskrause
Use blocking call to get corpus explorer content
1a36b35
@thomaskrause
Remove progress bar from corpus explorer window
2289c2c
@thomaskrause
Avoid storing reference to UI in result view panel
805a8e2
@thomaskrause
Visualizer creation background job needs to have a reference to the UI
4233d17
@thomaskrause thomaskrause removed their assignment Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

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