Skip to content

feat(evlog): add edgeUrl ingest support#110

Merged
HugoRCD merged 3 commits intoHugoRCD:mainfrom
gabrielelpidio:feat/axiom-edge-url
Feb 25, 2026
Merged

feat(evlog): add edgeUrl ingest support#110
HugoRCD merged 3 commits intoHugoRCD:mainfrom
gabrielelpidio:feat/axiom-edge-url

Conversation

@gabrielelpidio
Copy link
Contributor

@gabrielelpidio gabrielelpidio commented Feb 23, 2026

🔗 Linked issue

Resolves #46
Fixes EVL-47

📚 Description

This PR adds edge endpoint support to the Axiom adapter and aligns evlog’s config surface with the Axiom SDK direction.

Changes included:

  • Add edgeUrl support in createAxiomDrain() config resolution.
  • Add environment variable support for edge URL:
    • NUXT_AXIOM_EDGE_URL
    • AXIOM_EDGE_URL
  • Use mutually exclusive endpoint config typing (edgeUrl xor baseUrl) for adapter and Nuxt module options.
  • Implement ingest URL resolution for both endpoint types:
    • edgeUrl -> /v1/ingest/{dataset}
    • baseUrl (or default) -> /v1/datasets/{dataset}/ingest
  • Preserve custom edgeUrl paths as-is (trim trailing slash), matching upstream behavior.
  • Add runtime guard for mixed runtime/env config: when both are present, warn and prioritize edgeUrl.
  • Extend adapter tests to cover edge URL resolution and dataset encoding for edge ingest.
  • Update Axiom adapter docs with new env vars and endpoint option behavior.

Validation run locally:

  • bun run test test/adapters/axiom.test.ts
  • bun run build (package)
  • bun run lint src/adapters/axiom.ts src/nuxt/module.ts test/adapters/axiom.test.ts

📝 Checklist

  • I have linked an issue or discussion.
  • I have updated the documentation accordingly.

@vercel
Copy link

vercel bot commented Feb 23, 2026

@gabrielelpidio is attempting to deploy a commit to the HRCD Projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

Thank you for following the naming conventions! 🙏

@gabrielelpidio gabrielelpidio changed the title feat(axiom): add edgeUrl ingest support feat(evlog): add edgeUrl ingest support Feb 23, 2026
@HugoRCD
Copy link
Owner

HugoRCD commented Feb 24, 2026

@gabrielelpidio Thanks for the PR! Quick question, when edgeUrl has a custom path (e.g. https://edge.axiom.co/custom/ingest/), the dataset is not appended, only the trailing slash is stripped. Is that intentional? If so, could you document this behavior? The config table currently only mentions "Uses /v1/ingest/{dataset}" without this exception.

@vercel
Copy link

vercel bot commented Feb 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
evlog-docs Ready Ready Preview, Comment, Open in v0 Feb 24, 2026 1:07pm

@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 24, 2026

npm i https://pkg.pr.new/evlog@110
npm i https://pkg.pr.new/@evlog/nuxthub@110

commit: 0858a44

@gabrielelpidio
Copy link
Contributor Author

@gabrielelpidio Thanks for the PR! Quick question, when edgeUrl has a custom path (e.g. https://edge.axiom.co/custom/ingest/), the dataset is not appended, only the trailing slash is stripped. Is that intentional? If so, could you document this behavior? The config table currently only mentions "Uses /v1/ingest/{dataset}" without this exception.

Hey! Yes, it's intended, added some documentation around it

@HugoRCD HugoRCD merged commit 36f6913 into HugoRCD:main Feb 25, 2026
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[feature] edgeUrl support for Axiom adapter

3 participants