Skip to content

fix: allow multiple instances of GAP#1007

Merged
erikburt merged 3 commits into
mainfrom
fix/setup-gap-multiple-instances
Apr 22, 2025
Merged

fix: allow multiple instances of GAP#1007
erikburt merged 3 commits into
mainfrom
fix/setup-gap-multiple-instances

Conversation

@erikburt
Copy link
Copy Markdown
Contributor

@erikburt erikburt commented Apr 18, 2025

Changes

  • fix: (action) add --project parameter for docker compose commands
    • this should allow for multiple GAPs to be up'd
  • fix: (envoy config) websocket service routing with explicit port
    • this constrains the domain from any suffix, to a specific port suffix only
  • fix: (authz + envoy config) host header port rewrites
    • Now properly rewrites the host header to ensure a 443 port, which is what the upstream LBs are listening on. This only affected GAPs with non-standard ports
  • fix: (authz) logging level environment variable name
    • Fixes enabling debug logs for the authz service
  • feat: (authz) better header debug logging
    • more and better logging for

Testing


DX-541

@erikburt erikburt self-assigned this Apr 18, 2025
@erikburt erikburt force-pushed the fix/setup-gap-multiple-instances branch 15 times, most recently from 8c66ab8 to 38206bc Compare April 22, 2025 17:00
@erikburt erikburt force-pushed the fix/setup-gap-multiple-instances branch from 38206bc to c18a297 Compare April 22, 2025 17:07
@erikburt erikburt marked this pull request as ready for review April 22, 2025 17:18
@erikburt erikburt requested a review from a team as a code owner April 22, 2025 17:18
@erikburt erikburt merged commit 7182d39 into main Apr 22, 2025
18 checks passed
@erikburt erikburt deleted the fix/setup-gap-multiple-instances branch April 22, 2025 17:22
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