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

fix(service): omit nil endpoints and prefer endpointsForHostname() #4293

Merged
merged 3 commits into from
Mar 4, 2024
Merged

fix(service): omit nil endpoints and prefer endpointsForHostname() #4293

merged 3 commits into from
Mar 4, 2024

Commits on Mar 4, 2024

  1. fix(source): use NewEndpointWithTTL and omit nil Endpoints

    @yurrriq
    yurrriq committed Mar 4, 2024
    Configuration menu
    Copy the full SHA
    36a6e19 View commit details
    Browse the repository at this point in the history

  2. chore(ingress): add test with invalid hostname

    @yurrriq
    yurrriq committed Mar 4, 2024
    Configuration menu
    Copy the full SHA
    1eec428 View commit details
    Browse the repository at this point in the history

  3. fix(service): omit nil endpoints and prefer endpointsForHostname() 

    Also add a test with an invalid hostname.
    @yurrriq
    yurrriq committed Mar 4, 2024
    Configuration menu
    Copy the full SHA
    3fac88b View commit details
    Browse the repository at this point in the history