feat(core): run bootstrap hook after listen server. (#2795)

[underfin]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• [x ] Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: 2795

What is the new behavior?

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Other information

[underfin]

@hardcodet ，I think it as well. But something can be invoke with other hook, the onApplicationBootstrap hook should called after the server listened.

[kamilmysliwiec]

@hardcodet ，I think it as well. But something can be invoke with other hook, the onApplicationBootstrap hook should called after the server listened.

I'm not entirely sure about that. onApplicationBootstrap is called for other application types as well, like hybrid app or context app. This change would bring inconsistency between them.

[underfin]

OK, I will work it for other hook.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.