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

Auto register custom ITransportFactory in Spring integration. #1194

Merged
merged 5 commits into from
Jan 21, 2021

Conversation

maciejwalkowiak
Copy link
Contributor

📜 Description

Auto register custom ITransportFactory in Spring integration.

💡 Motivation and Context

For users who prefer to use Apache Http Client based transport, we have to provide convenient way of registering alternative transport factories.

💚 How did you test it?

Unit test.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing
  • No breaking changes

@codecov-io
Copy link

codecov-io commented Jan 21, 2021

Codecov Report

Merging #1194 (8a192e7) into main (fc69ce7) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #1194      +/-   ##
============================================
+ Coverage     74.70%   74.71%   +0.01%     
  Complexity     1677     1677              
============================================
  Files           176      176              
  Lines          5887     5890       +3     
  Branches        574      574              
============================================
+ Hits           4398     4401       +3     
  Misses         1225     1225              
  Partials        264      264              
Impacted Files Coverage Δ Complexity Δ
.../io/sentry/spring/SentryInitBeanPostProcessor.java 94.73% <100.00%> (+0.98%) 5.00 <0.00> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc69ce7...8a192e7. Read the comment docs.

@maciejwalkowiak maciejwalkowiak merged commit c002817 into main Jan 21, 2021
@maciejwalkowiak maciejwalkowiak deleted the spring-custom-transport-factory branch January 21, 2021 12:52
@Test
fun `configures custom TransportFactory`() {
ApplicationContextRunner().withConfiguration(UserConfigurations.of(AppConfigWithCustomTransportFactory::class.java))
.run {
Copy link
Member

Choose a reason for hiding this comment

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

This require a note on the docs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's included in getsentry/sentry-docs#2833

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.

4 participants