Skip to content

feat: GAP authz w/ docker compose#997

Merged
erikburt merged 14 commits into
mainfrom
feat/setup-gap-authz-docker-compose
Apr 16, 2025
Merged

feat: GAP authz w/ docker compose#997
erikburt merged 14 commits into
mainfrom
feat/setup-gap-authz-docker-compose

Conversation

@erikburt
Copy link
Copy Markdown
Contributor

@erikburt erikburt commented Apr 15, 2025

Changes

  • Switch from the LUA script to an authz ext filter (http service)
  • Use docker compose to up the envoy proxy and the authz http service
  • Fix DNS issue for dynamic proxy by forcing DNS resolution via DNS lookup

Testing

https://github.com/smartcontractkit/releng-test/pull/81 / https://github.com/smartcontractkit/releng-test/actions?query=branch%3Atest%2Fsetup-gap-authz-docker-compose

Note: The failure (https://github.com/smartcontractkit/releng-test/actions/runs/14498947677/job/40673702621?pr=81) is also present for runs on setup-gap 4.2, so not sure what the issue is there.


DX-541

@erikburt erikburt self-assigned this Apr 15, 2025
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
@erikburt erikburt force-pushed the feat/setup-gap-authz-docker-compose branch 7 times, most recently from 22d59b8 to 52e3a9f Compare April 15, 2025 21:54
Comment thread actions/setup-gap/authz/main.go Fixed
Comment thread actions/setup-gap/authz/main.go Fixed
@erikburt erikburt force-pushed the feat/setup-gap-authz-docker-compose branch 9 times, most recently from cd264c5 to e659b75 Compare April 15, 2025 23:40
@erikburt erikburt force-pushed the feat/setup-gap-authz-docker-compose branch from e659b75 to 8db8b8a Compare April 16, 2025 00:07
@erikburt erikburt mentioned this pull request Apr 16, 2025
@erikburt erikburt force-pushed the feat/setup-gap-authz-docker-compose branch 2 times, most recently from c868003 to 2212963 Compare April 16, 2025 19:50
@erikburt erikburt force-pushed the feat/setup-gap-authz-docker-compose branch from 2212963 to 2153566 Compare April 16, 2025 19:52
@erikburt erikburt marked this pull request as ready for review April 16, 2025 20:00
@erikburt erikburt requested a review from a team as a code owner April 16, 2025 20:00
Comment thread actions/setup-gap/docker-compose.yml Outdated
@erikburt erikburt merged commit f5ce463 into main Apr 16, 2025
18 checks passed
@erikburt erikburt deleted the feat/setup-gap-authz-docker-compose branch April 16, 2025 20:26
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.

3 participants