Skip to content

refactor: remove root-level organization_name config#810

Merged
alexluong merged 2 commits intomainfrom
refactor/remove-organization-name
Apr 8, 2026
Merged

refactor: remove root-level organization_name config#810
alexluong merged 2 commits intomainfrom
refactor/remove-organization-name

Conversation

@alexluong
Copy link
Copy Markdown
Collaborator

Summary

  • Remove OrganizationName config field (ORGANIZATION_NAME env var) from root-level config
  • Simplify User-Agent fallback: defaults to Outpost/{version} when HTTP_USER_AGENT is unset
  • Portal-level PORTAL_ORGANIZATION_NAME is unaffected

The field was only used as a fallback for generating the HTTP User-Agent string. Users who want a custom User-Agent should set HTTP_USER_AGENT directly.

Test plan

  • go build ./... passes
  • go test ./internal/config/... -count=1 passes

🤖 Generated with Claude Code

The OrganizationName field was only used as a fallback for generating the
HTTP User-Agent header when http_user_agent wasn't set. Users should use
http_user_agent directly instead. The portal-level PORTAL_ORGANIZATION_NAME
is unaffected.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 8, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
outpost-docs Ready Ready Preview, Comment Apr 8, 2026 1:04pm
outpost-website Ready Ready Preview, Comment Apr 8, 2026 1:04pm

Request Review

The e2e basic config never set Alert.AutoDisableDestination or
ConsecutiveFailureCount, so the opevents consecutive-failure test
timed out waiting for a destination disable that could never happen.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@alexluong alexluong merged commit e3bf6af into main Apr 8, 2026
5 checks passed
@alexluong alexluong deleted the refactor/remove-organization-name branch April 8, 2026 14:44
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