Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Documenting correlation package #558

Closed
Tracked by #534
mantzas opened this issue Sep 28, 2022 · 0 comments · Fixed by #589
Closed
Tracked by #534

Documenting correlation package #558

mantzas opened this issue Sep 28, 2022 · 0 comments · Fixed by #589
Assignees
@mantzas
Labels
maintenance Maintenance work
Milestone
v1.0.0 Release

Comments

@mantzas
Copy link

mantzas commented Sep 28, 2022

No description provided.

@mantzas mantzas mentioned this issue Sep 28, 2022
Open
15 tasks
@mantzas mantzas changed the title correlation Documenting correlation package Sep 28, 2022
@mantzas mantzas added enhancement New feature or request maintenance Maintenance work and removed enhancement New feature or request labels Sep 28, 2022
@mantzas mantzas added this to the v1.0.0 Release milestone Sep 28, 2022
@mantzas mantzas mentioned this issue Nov 27, 2022
Merged
@mantzas mantzas self-assigned this Nov 27, 2022
mantzas pushed a commit that referenced this issue Dec 1, 2022
Correlation doc and code improvement (#589)
a43c395 
<!--
Thanks for taking precious time for making a PR.

Before creating a pull request, please make sure:
- Your PR solves one problem for which an issue exist and a solution has
been discussed
- You have read the guide for contributing
- See
https://github.com/beatlabs/patron/blob/master/README.md#how-to-contribute
- You signed all your commits (otherwise we won't be able to merge the
PR)
  - See https://github.com/beatlabs/patron/blob/master/SIGNYOURWORK.md
- You added unit tests for the new functionality
- You mention in the PR description which issue it is addressing, e.g.
"Resolves #123"
-->

## Which problem is this PR solving?

Closes #558.

## Short description of the changes

- Moved HTTP correlation code to the middleware
- Moved HTTP correlation code to the legacy HTTP server which will be
deprecated soon
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mantzas mantzas

Labels
maintenance Maintenance work
Projects
None yet
Milestone
v1.0.0 Release
Development

Successfully merging a pull request may close this issue.

Correlation doc and code improvement beatlabs/patron
1 participant
@mantzas