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

fix(freeze): Remove/replace freezing request #1110

Merged

Conversation

domsteinbach
Copy link
Contributor

resolves DEV-1562

@linear
Copy link

linear bot commented Jun 22, 2023

DEV-1562 APP freezes completely if SIPI does not respond

It seems that there is a case when SIPI is not responding, APP hangs totally. It is related to components with an image browser (OpenSeaDragon). Improve the request handling, that it is not freezing the app while is waiting for SIPI response.

Because the issue is rather hard to reproduce, an idea would be to fake this behaviour by sleeping SIPI in LUA script.

@domsteinbach domsteinbach marked this pull request as ready for review June 22, 2023 08:33
@domsteinbach domsteinbach requested a review from mdelez June 22, 2023 08:33
* return the jwt token from the session to authenticate
* @return the token
*/
public getTokenFromLocalStorage(): string {
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is the same method that you have below but is private. Should it be public or private?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Damn, was due to merging. (I changed it to public in order the pdf viewer could possibly also send the token.)

@@ -10,7 +10,7 @@

<pdf-viewer
class="pdf-viewer"
[src]= "{ url: src.fileValue.fileUrl, withCredentials: true }"
[src]= "{ url: src.fileValue.fileUrl, withCredentials: true }"
Copy link
Collaborator

Choose a reason for hiding this comment

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

does withCredentials have a fear of commas? :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just needed some space ;-)

@domsteinbach domsteinbach merged commit 6a9e259 into main Jun 22, 2023
11 checks passed
@domsteinbach domsteinbach deleted the feature/dev-1562-app-freezes-if-sipi-does-not-respond branch June 22, 2023 09:34
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.

None yet

2 participants