Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix http proxy template #501

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

diedpigs
Copy link
Collaborator

@diedpigs diedpigs commented Jan 6, 2025

Replace hardcoded value with variable
Remove quote around the ood_user_regex so we can inject multiline easier
Remove unused code
Fix indentation

@jprorama
Copy link
Owner

jprorama commented Jan 6, 2025

This looks like clean up code. Let's hold off on merging that until we have a working proxy config.

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