Skip to content

Conversation

thanus
Copy link
Contributor

@thanus thanus commented Oct 3, 2020

When I wanted to use spring-boot-starter-mail I noticed that jakarta.mail should be used instead of javax.mail. This has now been changed in the documentation.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Oct 3, 2020
@@ -4421,10 +4421,10 @@ This section describes how to send email with the Spring Framework.
The following JAR needs to be on the classpath of your application in order to use
the Spring Framework's email library:

* The https://javaee.github.io/javamail/[JavaMail] library
* The https://eclipse-ee4j.github.io/mail/[JavaMail] library
Copy link
Contributor Author

@thanus thanus Oct 3, 2020

Choose a reason for hiding this comment

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

It seems like Jakarta Mail is the new name for JavaMail. Should this also be changed? JavaMail is also mentioned in several places (docs, codebase).

Copy link
Contributor

Choose a reason for hiding this comment

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

We could mention both names. JavaMail is still the canonical name for the API in Java EE 7/8 and therefore our baseline. Jakarta EE 9 and Jakarta Mail are not common enough (yet) to switch the primary names that we use for those APIs.

@sdeleuze sdeleuze self-assigned this Nov 10, 2020
@sdeleuze sdeleuze added type: documentation A documentation task and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Nov 10, 2020
@sdeleuze sdeleuze added this to the 5.3.2 milestone Nov 10, 2020
@jhoeller jhoeller assigned jhoeller and unassigned sdeleuze Nov 17, 2020
@jhoeller jhoeller merged commit 9139cb8 into spring-projects:master Nov 17, 2020
jhoeller added a commit that referenced this pull request Nov 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: documentation A documentation task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants