Skip to content

Conversation

@thesandlord
Copy link
Contributor

Description

Refs: Slack thread from @thesandlord about fern generate --docs stalling in air-gapped environments

Link to Devin run: https://app.devin.ai/sessions/dcec6a094378444b94e7253cc9c06cf4
Requested by: Sandeep Dinesh (@thesandlord)

In self-hosted environments, the air-gap detection was checking connectivity to https://registry.buildwithfern.com/health (from DEFAULT_FDR_ORIGIN) instead of the local FDR server at http://localhost:8080/health (from OVERRIDE_FDR_ORIGIN). This caused the CLI to incorrectly detect air-gapped mode even when the local FDR server was reachable.

The FDR client already respects OVERRIDE_FDR_ORIGIN, but the air-gap detection did not.

Changes Made

  • Updated publishDocs.ts and runRemoteGenerationForGenerator.ts to use OVERRIDE_FDR_ORIGIN ?? DEFAULT_FDR_ORIGIN for air-gap detection, matching the FDR client behavior
  • Improved debug logging for all air-gap checks to show:
    • The URL being tested
    • The timeout value
    • The result (connected/air-gapped) with error details when applicable
  • Added changelog entry for version 3.51.3

Testing

  • Manual testing in air-gapped environment (requires self-hosted deployment)
  • Lint checks pass

Human Review Checklist

  • Verify the environment variable precedence (OVERRIDE_FDR_ORIGIN ?? DEFAULT_FDR_ORIGIN ?? fallback) is correct
  • Confirm no other files need this fix (searched for DEFAULT_FDR_ORIGIN usage in air-gap detection)

…ironments

Co-Authored-By: Sandeep Dinesh <sandeep@buildwithfern.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration devin-ai-integration bot changed the title fix: use OVERRIDE_FDR_ORIGIN for air-gap detection in self-hosted environments fix(cli): use OVERRIDE_FDR_ORIGIN for air-gap detection in self-hosted environments Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants