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

refactor: add 'ports' dict to the data structure #2410

Merged
merged 1 commit into from Mar 12, 2024

Conversation

pini-gh
Copy link
Contributor

@pini-gh pini-gh commented Mar 9, 2024

Nothing changes for the legacy syntax, but adding this level to the data structure enables advanced port configuration.

Nothing changes for the legacy syntax, but adding this level to the data
structure enables advanced port configuration.
@buchdag buchdag added the scope/multiport Issue or PR related to multi port proxying label Mar 12, 2024
@buchdag
Copy link
Member

buchdag commented Mar 12, 2024

Pretty much what I had in mind next, thank you 👍

@buchdag buchdag merged commit 94fb845 into nginx-proxy:main Mar 12, 2024
2 checks passed
@pini-gh pini-gh deleted the pini-refactor-template-3 branch May 9, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope/multiport Issue or PR related to multi port proxying
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants