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

launcher exits the VM with status code 0 when an exception is thrown VertxApplicationHooks #15

Open
bitkid opened this issue Dec 9, 2024 · 7 comments
Assignees
Labels
invalid This doesn't seem right

Comments

@bitkid
Copy link

bitkid commented Dec 9, 2024

if i put a throw Exception("bla") into one of the application hooks (f.ex. in afterVertxStarted) the JVM exits with status 0. i think it should not.

@bitkid bitkid added the bug Something isn't working label Dec 9, 2024
@vietj vietj added this to the 5.0.0 milestone Dec 10, 2024
@tsegismont tsegismont self-assigned this Jan 3, 2025
@tsegismont tsegismont added enhancement New feature or request and removed bug Something isn't working labels Jan 3, 2025
@tsegismont
Copy link
Contributor

Thanks for reporting this @bitkid

@tsegismont
Copy link
Contributor

@bitkid I've added a couple of tests in 76672e4 and couldn't reproduce your issue.

Can you share a small snippet that fails your expectations with 5.0.0.CR3?

@tsegismont tsegismont removed this from the 5.0.0 milestone Jan 15, 2025
@tsegismont tsegismont added invalid This doesn't seem right and removed enhancement New feature or request labels Jan 15, 2025
@tsegismont tsegismont closed this as not planned Won't fix, can't repro, duplicate, stale Jan 15, 2025
@bitkid
Copy link
Author

bitkid commented Jan 21, 2025

this is not working

@tsegismont
Copy link
Contributor

Please share a small snippet that fails your expectations with 5.0.0.CR3

@bitkid
Copy link
Author

bitkid commented Jan 21, 2025

https://gist.github.com/bitkid/30456f73833b9074e3ac39704256d4e2

this is the simple test code.

i put it into a fat jar and start it with

java -cp the-all.jar Launchie

it writes

"throw exception" in the shell

and when i do echo $? it returns 0

@bitkid
Copy link
Author

bitkid commented Jan 21, 2025

ok, so i looked at your tests .. do i have to do

System.exit(app.launch()) ?

if so, you can just close the ticket and document it somewhere?

@bitkid
Copy link
Author

bitkid commented Jan 21, 2025

ah ok .. System.exit() is only called when the exit code is 2 .. what's the reason behind that? because if the beforeStartingVertx throws exit code is 1 @tsegismont

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

3 participants