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.

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

feat: Add the MaxTagValueLength config for jaeger of tracing #2482

Merged
merged 2 commits into from
Apr 23, 2021

Conversation

kobayashilin
Copy link
Contributor

Related issue

#2447

Proposed changes

Integration with this change feat: [Jaeger] Add maxTagValueLength for the feature request #2447 and make it configurable in ory/hydra.

Checklist

  • I have read the contributing guidelines.
  • I have read the security policy.
  • I confirm that this pull request does not address a security
    vulnerability. If this pull request addresses a security. vulnerability, I
    confirm that I got green light (please contact
    security@ory.sh) from the maintainers to push
    the changes.
  • I have added tests that prove my fix is effective or that my feature
    works.
  • I have added or changed the documentation.

@kobayashilin kobayashilin changed the title feat. Add the MaxTagValueLength config for jaeger of tracing feat: Add the MaxTagValueLength config for jaeger of tracing Apr 22, 2021
@aeneasr aeneasr merged commit 03c96ee into ory:master Apr 23, 2021
mitar pushed a commit to mitar/hydra that referenced this pull request May 13, 2021
Closes ory#2447

Co-authored-by: hackerman <3372410+aeneasr@users.noreply.github.com>
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