Skip to content

공공api timeout 처리#3014

Merged
chanlee merged 2 commits into
masterfrom
develop
Sep 27, 2025
Merged

공공api timeout 처리#3014
chanlee merged 2 commits into
masterfrom
develop

Conversation

@chanlee
Copy link
Copy Markdown
Collaborator

@chanlee chanlee commented Sep 27, 2025

No description provided.

@chanlee chanlee merged commit 9c4b9ae into master Sep 27, 2025
4 checks passed
Comment thread src/util/common.js
const response = await fetch(`${Entry.baseUrl || ''}${options.url}${queryString}`, options);
if (response.status >= 400) {
const fetchPromise = fetch(`${Entry.baseUrl || ''}${options.url}${queryString}`, options);
const timeoutPromise = new Promise((_, reject) =>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚫 [eslint] <prettier/prettier> reported by reviewdog 🐶
Delete ·

Suggested change
const timeoutPromise = new Promise((_, reject) =>
const timeoutPromise = new Promise((_, reject) =>

Comment thread src/util/common.js
if (response.status >= 400) {
const fetchPromise = fetch(`${Entry.baseUrl || ''}${options.url}${queryString}`, options);
const timeoutPromise = new Promise((_, reject) =>
setTimeout(() => reject(new Error('Request timeout')), timeout)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] <no-undef> reported by reviewdog 🐶
'timeout' is not defined.

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.

1 participant