Skip to content

fix: fail immediately when --url is unreachable instead of 60s timeout @W-21331028@#31

Merged
deepu-mungamuri94 merged 1 commit intomainfrom
fix/url-flag-issue
Mar 4, 2026
Merged

fix: fail immediately when --url is unreachable instead of 60s timeout @W-21331028@#31
deepu-mungamuri94 merged 1 commit intomainfrom
fix/url-flag-issue

Conversation

@deepu-mungamuri94
Copy link
Copy Markdown
Collaborator

  • When --url is provided and unreachable, fail immediately with clear error
  • Add error.dev-url-unreachable-with-flag message for --url case
  • Suggests removing --url to use dev.command when user has manifest config

What does this PR do?

This PR updates sf webapp dev so that when --url is passed with an unreachable URL, the command fails immediately instead of starting the dev server and waiting 60 seconds for a timeout. It also adds a clearer error message for this case that suggests removing --url to use dev.command when the manifest already has dev config.

What issues does this PR fix or reference?

@W-21331028@

…t @W-21331028@

- When --url is provided and unreachable, fail immediately with clear error
- Add error.dev-url-unreachable-with-flag message for --url case
- Suggests removing --url to use dev.command when user has manifest config

Made-with: Cursor
@deepu-mungamuri94 deepu-mungamuri94 requested a review from a team as a code owner March 4, 2026 10:40
Copy link
Copy Markdown
Collaborator

@amritmishra-sf amritmishra-sf left a comment

Choose a reason for hiding this comment

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

LGTM

@deepu-mungamuri94 deepu-mungamuri94 merged commit 5dc12d6 into main Mar 4, 2026
16 checks passed
@deepu-mungamuri94 deepu-mungamuri94 deleted the fix/url-flag-issue branch March 4, 2026 12:17
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.

2 participants