Skip to content

Auto-configuration for SendGrid's Java client #2280

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

Closed

Conversation

maciejwalkowiak
Copy link
Contributor

Pull for #2160. SendGrid gets auto-configured if:

  • com.sendgrid.SendGrid is on classpath
  • property: sendgrid.username is defined
  • additionally auto-configures proxy when sendgrid.proxy.host and sendgrid.proxy.port is defined

Documentation - only Appendix A. Common application properties section has been updated with new properties.

Unit tests - testing proxy creation could be improved if I could set up some mock HTTP server (for example with Jadler) and see if there are requests coming in. At the moment there is a check if SendGrid bean is created but there is no really a way to check if it's created with proxy or not.

@philwebb philwebb added the type: enhancement A general enhancement label Jan 5, 2015
@philwebb
Copy link
Member

philwebb commented Jan 5, 2015

Nice, we'll take a look at this when we switch master to 1.3.

@philwebb philwebb added this to the 1.3.0 milestone Jan 5, 2015
@snicoll
Copy link
Member

snicoll commented Jan 6, 2015

This may sound a bit repetitive but could we please avoid using root context unless we really have to? spring.sendgrid is a bit longer admittedly but it avoids taking yet another root namespace.

@philwebb
Copy link
Member

philwebb commented Jan 6, 2015

+1 for the property name change

@wilkinsona wilkinsona closed this in f05769d Mar 5, 2015
@wilkinsona
Copy link
Member

Thanks for the PR. I've made one notable change: the properties are now spring.sendgrid.* rather than sendgrid.*

@maciejwalkowiak
Copy link
Contributor Author

Cool, thanks!

@snicoll
Copy link
Member

snicoll commented Mar 5, 2015

👍

@wilkinsona wilkinsona modified the milestones: 1.3.0, 1.3.0.M1 May 14, 2015
@patrickbray
Copy link

This is great are there any plans to extend this to support SendGrid API keys as well?

@wilkinsona
Copy link
Member

I don't believe so. Please open a new issue or, even better, a pull request if that's an enhancement that you would like to be made.

@patrickbray
Copy link

👍 will do

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants