Skip to content

ngx_otel_module support for HTTPS #963

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

Open
kevinkirkup opened this issue Feb 5, 2025 · 3 comments
Open

ngx_otel_module support for HTTPS #963

kevinkirkup opened this issue Feb 5, 2025 · 3 comments

Comments

@kevinkirkup
Copy link

Describe the bug

nix_otel_module OTEL exporter plugin has a bug where it doesn't support HTTPS.

It's fixed in their main branch, but not in the version distributed in the latest docker image.

To reproduce

Steps to reproduce the behavior:

  1. Deploy the NGINX Docker image using:

    otel_exporter {
        endpoint "https://otel.labt.fp.f5net.com:4433";
    
        trusted_certificate “path/to/custom/ca/bundle“; # optional
    }
    
  2. View logs

  3. See error

    OTel export failure: DNS resolution failed for https://xyz.com:4317: C-ares status is not ARES_SUCCESS qtype=A name=https://otel-collector.ksk.opdt.cc:4317 is_balancer=0: Misformatted domain name
    

Expected behavior

Should connect

Your environment

  • Version/release of Docker and method of installation (e.g. Docker Desktop / Docker Server)
    Any

  • Version/tag of the NGINX Docker image (e.g. nginx:alpine)
    I'm using Alpine, but affects all image versions.

  • Target deployment platform
    I'm using AWS ECS but impacts all.

Additional context

Here is a link to the issue in the module repository w/ details.

nginxinc/nginx-otel#12

Fixed Dec 24, 2024

@kevinkirkup
Copy link
Author

Upstream Repo needs a new tag.
It also looks like the current docker images is missing at least a patch version of the upstream module.
Latest is v0.1.1

@thresheek
Copy link
Member

Hi @kevinkirkup!

Packaging we use to feed the docker images actually already has this otel version included: nginx/pkg-oss@1b00f76

Docker image bump is going to happen in a couple of days with a new release of nginx (and thus modules too), so stay tuned!

@thresheek
Copy link
Member

The updated nginx:alpine-otel is up. Can you please test @kevinkirkup ? Thanks!

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

No branches or pull requests

2 participants