Skip to content

Commit

Permalink
Req 1.27 SemConv (#375)
Browse files Browse the repository at this point in the history
* Req 1.26 SemConv

* bump semconv

* Format

* otel http

* sem conv in main

* update deps

* Update deps

* Update instrumentation/opentelemetry_req/lib/opentelemetry_req.ex

Co-authored-by: Greg Mefford <[email protected]>

* Update instrumentation/opentelemetry_req/lib/opentelemetry_req.ex

Co-authored-by: Greg Mefford <[email protected]>

* Guard against multiple headers

* Bump otel

* moar pipes

* Update docs around path params

* Remove unused opt

* Align opt-in attrs with other HTTP libs

* Make RC and update changelog

* Dialyzer and fixes

---------

Co-authored-by: Greg Mefford <[email protected]>
  • Loading branch information
bryannaegele and GregMefford authored Oct 16, 2024
1 parent 98d7010 commit 3f43ca2
Show file tree
Hide file tree
Showing 6 changed files with 724 additions and 156 deletions.
27 changes: 19 additions & 8 deletions instrumentation/opentelemetry_req/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,32 +1,43 @@
# Changelog

### 1.0.0-rc.1

### Features

- OpenTelemetry v1.27 support

### Breaking Changes

- Various HTTP Semantic Convention changes are included. One major change
regards span naming. This may affect your observability tools when keying
on span names. The key change there is the HTTP method is now a prefix, e.g. "GET /users/:user_id"

## 0.2.0

### Fixes

* Add support for Req v0.4
- Add support for Req v0.4

* Change http.url to follow [OpenTelemetry http spec](https://opentelemetry.io/docs/reference/specification/trace/semantic_conventions/http/#http-client).
- Change http.url to follow [OpenTelemetry http spec](https://opentelemetry.io/docs/reference/specification/trace/semantic_conventions/http/#http-client).

* Full HTTP request URL in the form `scheme://host[:port]/path?query[#fragment]`
- Full HTTP request URL in the form `scheme://host[:port]/path?query[#fragment]`

* Strip user credentials passed via URL
- Strip user credentials passed via URL

## 0.1.2

### Fixes

* Fix ctx not being set back to parent upon completion
- Fix ctx not being set back to parent upon completion

## 0.1.1

### Fixes

* Fix client span to be the ctx injected to headers
- Fix client span to be the ctx injected to headers

## 0.1.0

### Features

* Initial release

- Initial release
2 changes: 1 addition & 1 deletion instrumentation/opentelemetry_req/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ See [Docs](https://hex.pm/packages/opentelemetry_req) for usage instructions.
```elixir
def deps do
[
{:opentelemetry_req, "~> 0.2.0"}
{:opentelemetry_req, "~> 1.0.0-beta.1"}
]
end
```
Expand Down
Loading

0 comments on commit 3f43ca2

Please sign in to comment.