Skip to content

Update URL transformations for new hostname patterns#7

Merged
jkebinger merged 2 commits into
mainfrom
update-url-transformations
Sep 19, 2025
Merged

Update URL transformations for new hostname patterns#7
jkebinger merged 2 commits into
mainfrom
update-url-transformations

Conversation

@jkebinger
Copy link
Copy Markdown
Contributor

Summary

  • Updated URL transformation logic to support both old and new hostname patterns
  • Telemetry URLs: Now transforms both primary/secondary and belt/suspenders to telemetry.
  • SSE Stream URLs: Now transforms both primary/secondary and belt/suspenders to stream.
  • Maintains backward compatibility with existing belt/suspenders hostnames
  • Works with new primary/secondary hostnames (e.g., primary.goatsofreforge.comstream.goatsofreforge.com)

Changes

  • lib/reforge/options.rb: Updated telemetry destination transformation
  • lib/reforge/sse_config_client.rb: Updated SSE stream URL transformation

Test plan

  • Existing tests pass with new transformation logic
  • Both old and new hostname patterns are supported
  • URL transformations correctly generate telemetry and stream subdomains

🤖 Generated with Claude Code

jkebinger and others added 2 commits September 19, 2025 17:19
Support both old (belt/suspenders) and new (primary/secondary) hostname patterns
for telemetry and stream URL transformations:

- Update telemetry destination transformation in Options to handle primary/secondary
- Update SSE stream URL transformation to handle primary/secondary
- Maintain backward compatibility with belt/suspenders hostnames

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Update integration and SSE client tests to use the new hostname patterns:
- Replace belt/suspenders.staging-prefab.cloud with primary/secondary.goatsofreforge.com
- Update api.staging-prefab.cloud to api.goatsofreforge.com
- Ensure test expectations match new URL transformation logic

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jkebinger jkebinger force-pushed the update-url-transformations branch from ef2a499 to ed9bfaf Compare September 19, 2025 22:19
Copy link
Copy Markdown
Contributor

@jdwyah jdwyah left a comment

Choose a reason for hiding this comment

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

thanks

@jkebinger jkebinger merged commit 8353239 into main Sep 19, 2025
4 checks passed
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