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

WV-3143: Adding Client Side Timeout to EIC Request #5105

Merged
merged 1 commit into from
Apr 15, 2024
Merged

WV-3143: Adding Client Side Timeout to EIC Request #5105

merged 1 commit into from
Apr 15, 2024

Conversation

ryanweiler92
Copy link
Collaborator

Description

Adding a 10 second client side timeout to the EIC request to handle API GW and other DNS issues.

How To Test

  1. git checkout wv-3143
  2. npm run watch
  3. Open the eic.js file.
  4. Delete a few characters from the hostname on line 19 of the URL
  5. Open this EIC URL
  6. Verify that it still moves the date via the legacy mode by opening up the console and viewing the print statements. It should have various statements like "Entering EIC mode". These statements do not appear outside of legacy mode.

@nasa-gibs/worldview

@ryanweiler92 ryanweiler92 added this to the v4.32.0 milestone Apr 10, 2024
@ryanweiler92 ryanweiler92 self-assigned this Apr 10, 2024
@ryanweiler92 ryanweiler92 merged commit 2e3323d into develop Apr 15, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@PatchesMaps PatchesMaps PatchesMaps approved these changes

@Tomcariello Tomcariello Tomcariello approved these changes

@christof-wittreich christof-wittreich Awaiting requested review from christof-wittreich

@minniewong minniewong Awaiting requested review from minniewong

Assignees

@ryanweiler92 ryanweiler92

Labels
None yet
Projects
None yet
Milestone
v4.32.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ryanweiler92 @PatchesMaps @Tomcariello