Skip to content

fix: fix latest sanic release note #2703

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
Merged
merged 1 commit into from
Jul 27, 2021
Merged

fix: fix latest sanic release note #2703

merged 1 commit into from
Jul 27, 2021

Conversation

P403n1x87
Copy link
Contributor

Description

Checklist

  • Added to the correct milestone.
  • Tests provided or description of manual testing performed is included in the code or PR.
  • Library documentation is updated.
  • Corp site documentation is updated (link to the PR).

@P403n1x87 P403n1x87 added the changelog/no-changelog A changelog entry is not required for this PR. label Jul 27, 2021
@P403n1x87 P403n1x87 requested a review from a team as a code owner July 27, 2021 13:47
@brettlangdon brettlangdon merged commit 5367d43 into DataDog:master Jul 27, 2021
Kyle-Verhoog pushed a commit that referenced this pull request Jul 28, 2021
@P403n1x87 @Kyle-Verhoog
fix: fix latest sanic release note (#2703)
fc77d61
mergify bot added a commit that referenced this pull request Jul 28, 2021
@mergify @P403n1x87
fix(sanic): fix handling of non-string arguments (backport #2699) (#2708
27f83db 
)

* fix(sanic): fix handling of non-string arguments (#2699)

* fix(sanic): fix handling of non-string arguments

This change fixes the handling of endpoint paths that take non-string
arguments.

Fixes #2678.

* Update ddtrace/contrib/sanic/patch.py

Co-authored-by: Nicolas Vivet <[email protected]>

Co-authored-by: Nicolas Vivet <[email protected]>
(cherry picked from commit c95322a)

* fix: fix latest sanic release note (#2703)

Co-authored-by: Gabriele N. Tornetta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@brettlangdon brettlangdon brettlangdon approved these changes

@Kyle-Verhoog Kyle-Verhoog Kyle-Verhoog approved these changes

Assignees
No one assigned
Labels
changelog/no-changelog A changelog entry is not required for this PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@P403n1x87 @brettlangdon @Kyle-Verhoog