Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

bug: senders.NewSender does not support urls with both port and path. #108

@LukeWinikates

Description

@LukeWinikates

This was originally reported as a bug on the wavefront telegraf plugin: influxdata/telegraf#11925

senders.CreateConfig("http://localhost:8071/wavefront") currently removes the port portion of the url during the parsing process. This doesn't create any problems for urls without a path component, but when port and path are both present, the resulting url is mangled: "http://localhost/wavefront", and the port is then appended by the reporter later, resulting in something like "http://localhost/wavefront:8071/report?f=wavefront".

I'm hoping to submit a PR with test cases in the next couple of hours.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions