Skip to content

Add cURL export feature on trace details page#146

Merged
georgidhristov merged 2 commits into
DebugProbe:mainfrom
DevSars24:feature/curl-export
Jul 9, 2026
Merged

Add cURL export feature on trace details page#146
georgidhristov merged 2 commits into
DebugProbe:mainfrom
DevSars24:feature/curl-export

Conversation

@DevSars24

Copy link
Copy Markdown
Contributor

No description provided.

@DevSars24 DevSars24 requested a review from georgidhristov as a code owner July 7, 2026 12:45
@georgidhristov

Copy link
Copy Markdown
Collaborator

Tested locally on Windows (cmd.exe) and found an issue with the generated cURL command.

The export currently uses single quotes (') around the URL and headers. Windows Command Prompt does not treat single quotes as string delimiters, so the command is parsed incorrectly and curl attempts to resolve header values as hostnames.

For example, the generated command results in errors such as:

curl: (6) Could not resolve host: text
curl: (3) URL rejected: Bad hostname

I recommend either:

  • Generating a Windows-compatible command using double quotes ("), or
  • Providing separate export formats for Bash/Linux/macOS and Windows (cmd.exe / PowerShell).

Everything else looks good, but this should be fixed before merging.

@georgidhristov

Copy link
Copy Markdown
Collaborator

Tested locally and everything works as expected.
Nice addition to DebugProbe. Approved and merged.

@georgidhristov georgidhristov merged commit 01596ed into DebugProbe:main Jul 9, 2026
1 check passed
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

Successfully merging this pull request may close these issues.

2 participants