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.

Sign up for GitHub

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

Jump to bottom

Added support for ddns lookups for addresses in access lists #3364

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Added support for ddns lookups for addresses in access lists #3364

wants to merge 8 commits into from

Commits on Apr 29, 2024

  1. Initial pass at DDNS support for client addresses 

    This is a first pass attempt at adding support for using ddns (really any resolvable domain name) as the address in access list clients.

This helps make it possible to restrict access to hosts using a dynamic public IP (e.g. allow access to a proxied host from your local network only via ddns address).

Current approach is hacky since it was developed by manually replacing files in an existing npm docker container. Future commits will integrate this better and avoid needing to patch/intercept existing APIs.

See associated PR for more details.
    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    5586709 View commit details
    Browse the repository at this point in the history

  2. Refactor and integrate ddns resolution with nginx module 

    Refactored ddns resolver so that no patching is done. nginx.js will automatically resolve ddns addresses if needed.

Added dedicated logger scope for ddns resovler.
    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    ec9eb0d View commit details
    Browse the repository at this point in the history

  3. fix linter warnings

    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    972d158 View commit details
    Browse the repository at this point in the history

  4. Fix utils.js linter error

    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    33f41f7 View commit details
    Browse the repository at this point in the history

  5. Eliminate circular dependency

    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    743cdd8 View commit details
    Browse the repository at this point in the history

  6. Update configs for active hosts only on ddns update 

    Other changes:
- Fixed null property read error on clients (when switching to public access)
- Use separate `resolvedAddress` field for resolved IP instead of overwriting address
- Reduced ddns log verbosity
    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    7b09fef View commit details
    Browse the repository at this point in the history

  7. doc string update

    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    3b0ff57 View commit details
    Browse the repository at this point in the history

  8. Add support for '-' in ddns domain names

    @vari
    vari committed Apr 29, 2024
    Configuration menu
    Copy the full SHA
    e317900 View commit details
    Browse the repository at this point in the history