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

Throwing Uncaught ReferenceError window when initialising the swagger client #10694

Open
tmkasun opened this issue Apr 1, 2021 · 1 comment
Labels
API-M 4.0.0 Related to 4.0.0 release of API-M Priority/Low React-UI Type/Bug UI-Performance React UI performance related issues

Comments

@tmkasun
Copy link
Contributor

tmkasun commented Apr 1, 2021

Description:

We have added a web worker to support parallelize the SwaggerClient initialization work during the page load.

But due to an update in SwaggerClient, we are getting the below exception when loading the client inside the worker thread

image

This exception simply break the whole API invocation flow hence we have fallen back to the old swagger client initialization mechanism

Raised a support issue in swagger-api/swagger-js#2008

We, Will, use this issue in the WSO2 end to track the progress, Until it gets resolved we will comment out the worker initialization code.

Impact:

Users will experience a slight delay in loading the publisher web app. Because Swagger resolver takes considerable time parsing the publisher OAS definition

Steps to reproduce:

  • Visit /publisher while the Developer console is open
@tmkasun tmkasun added Type/Bug Priority/Normal React-UI UI-Performance React UI performance related issues API-M 4.0.0 Related to 4.0.0 release of API-M labels Apr 1, 2021
@tmkasun tmkasun changed the title Throwing Uncaught ReferenceError window when initialising swagger client Throwing Uncaught ReferenceError window when initialising the swagger client Apr 1, 2021
@malinthaprasan
Copy link
Contributor

@tmkasun are we fixing this in 4.0?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API-M 4.0.0 Related to 4.0.0 release of API-M Priority/Low React-UI Type/Bug UI-Performance React UI performance related issues
Projects
None yet
Development

No branches or pull requests

2 participants