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

fix spring-webflux cast to PathPattern throws ClassCastException #6872

Merged

Conversation

wang007
Copy link
Contributor

@wang007 wang007 commented Oct 13, 2022

when ServerWebExchange.getAttributes().put(HandlerMapping.BEST_MATCHING_PATTERN_ATTRIBUTE, "path string type").
opentelemetry javaagent will throw exception.
企业微信截图_76344afd-4541-482c-a90f-0606ad720351

@wang007 wang007 requested a review from a team October 13, 2022 13:50
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Oct 13, 2022

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: wang007 / name: 王展城 (70e9d51)

@wang007
Copy link
Contributor Author

wang007 commented Oct 13, 2022

/easycla

@wang007 wang007 force-pushed the fix-webflux-cast-exception branch 3 times, most recently from 70e9d51 to 727bb49 Compare October 13, 2022 14:51
Copy link
Member

@trask trask left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx!

@mateuszrzeszutek
Copy link
Member

mateuszrzeszutek commented Oct 14, 2022

Hey @wang007 ,
Can you run ./gradlew spotlessApply and commit? That should fix the failing spotless job. Thanks!

@wang007 wang007 force-pushed the fix-webflux-cast-exception branch from 727bb49 to 29ce07e Compare October 21, 2022 03:03
@wang007
Copy link
Contributor Author

wang007 commented Oct 21, 2022

Hey @mateuszrzeszutek
done.

@mateuszrzeszutek
Copy link
Member

Thanks!

@mateuszrzeszutek mateuszrzeszutek merged commit 397a5a8 into open-telemetry:main Oct 21, 2022
dmarkwat pushed a commit to dmarkwat/opentelemetry-java-instrumentation that referenced this pull request Oct 22, 2022
…n-telemetry#6872)

when
ServerWebExchange.getAttributes().put(HandlerMapping.BEST_MATCHING_PATTERN_ATTRIBUTE,
"path string type").
opentelemetry javaagent will throw exception.

[企业微信截图_76344afd-4541-482c-a90f-0606ad720351](https://user-images.githubusercontent.com/15957476/195615160-45559fb5-5e0c-4c25-8678-7d8aa603f346.png)
@wang007
Copy link
Contributor Author

wang007 commented Oct 24, 2022

hey, @trask
when this commit release ?

@mateuszrzeszutek
Copy link
Member

This fix will be included in the 1.20 release, which is scheduled to happen around 16th Nov

LironKS pushed a commit to helios/opentelemetry-java-instrumentation that referenced this pull request Oct 31, 2022
…n-telemetry#6872)

when
ServerWebExchange.getAttributes().put(HandlerMapping.BEST_MATCHING_PATTERN_ATTRIBUTE,
"path string type").
opentelemetry javaagent will throw exception.

[企业微信截图_76344afd-4541-482c-a90f-0606ad720351](https://user-images.githubusercontent.com/15957476/195615160-45559fb5-5e0c-4c25-8678-7d8aa603f346.png)
LironKS pushed a commit to helios/opentelemetry-java-instrumentation that referenced this pull request Dec 4, 2022
…n-telemetry#6872)

when
ServerWebExchange.getAttributes().put(HandlerMapping.BEST_MATCHING_PATTERN_ATTRIBUTE,
"path string type").
opentelemetry javaagent will throw exception.

[企业微信截图_76344afd-4541-482c-a90f-0606ad720351](https://user-images.githubusercontent.com/15957476/195615160-45559fb5-5e0c-4c25-8678-7d8aa603f346.png)
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.

3 participants