Skip to content

feat: add test setup with dex & oauth2-proxy#1028

Merged
perber merged 1 commit into
mainfrom
feature/add-proxy-test-setup
May 24, 2026
Merged

feat: add test setup with dex & oauth2-proxy#1028
perber merged 1 commit into
mainfrom
feature/add-proxy-test-setup

Conversation

@perber
Copy link
Copy Markdown
Owner

@perber perber commented May 24, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 24, 2026 01:34
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a local developer “proxy auth” stack (Dex + oauth2-proxy) to exercise LeafWiki’s --enable-http-remote-user mode, including a variant that demonstrates proper logout behavior, and introduces a .dockerignore to slim Docker build contexts.

Changes:

  • Add two Docker Compose variants to run LeafWiki behind oauth2-proxy + Dex (with and without a configured logout URL).
  • Add a helper run.sh script to start/stop the stacks.
  • Add a .dockerignore to reduce Docker build context size (excluding e2e tooling, hacks, etc.).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
hacks/proxy-auth/with-logout.yml Compose stack wiring Dex + oauth2-proxy + LeafWiki with LeafWiki logout redirected to oauth2-proxy signout.
hacks/proxy-auth/without-logout.yml Same stack without LeafWiki logout redirect, to reproduce “can’t really log out” behavior.
hacks/proxy-auth/run.sh Convenience script to bring stacks up/down.
hacks/proxy-auth/dex/config.yaml Dex in-memory OIDC config + a static dev user/client for local testing.
.dockerignore Excludes non-build inputs (e2e, hacks, docs, etc.) from Docker build context.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread hacks/proxy-auth/with-logout.yml
Comment thread hacks/proxy-auth/with-logout.yml
Comment thread hacks/proxy-auth/with-logout.yml
Comment thread hacks/proxy-auth/without-logout.yml
Comment thread hacks/proxy-auth/without-logout.yml
Comment thread hacks/proxy-auth/without-logout.yml
Comment thread hacks/proxy-auth/run.sh
@perber perber merged commit 7c8f78f into main May 24, 2026
7 checks passed
@perber perber deleted the feature/add-proxy-test-setup branch May 24, 2026 01:40
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