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

Spring Boot CLI is unable to always capture ctrl-c #17976

Closed
wants to merge 1 commit into from
Closed

Spring Boot CLI is unable to always capture ctrl-c #17976

wants to merge 1 commit into from

Conversation

nebhale
Copy link
Member

@nebhale nebhale commented Aug 27, 2019

Previously, when the Spring Boot CLI ran the Java command line for an application it did a straight invocation which suppressed propagation of signals to the JVM in certain circumstances (e.g. within Docker containers).

This change prepends the command with exec which causes the Java process to replace the script process in a shell ensuring that signals are propagated to the Java process without suppression.

Previously, when the Spring Boot CLI ran the Java command line for an
application it did a straight invocation which suppressed propagation of
signals to the JVM in certain circumstances (e.g within Docker containers).

This change prepends the command with exec which causes the Java process to
replace the script process in a shell ensuring that signals are propagated to
the Java process without suppression.

Signed-off-by: Ben Hale <bhale@pivotal.io>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 27, 2019
@philwebb philwebb added for: team-attention An issue we'd like other members of the team to review type: task A general task and removed for: team-attention An issue we'd like other members of the team to review status: waiting-for-triage An issue we've not yet triaged labels Aug 28, 2019
@philwebb philwebb added this to the 2.1.x milestone Aug 28, 2019
@philwebb philwebb added type: bug A general bug and removed type: task A general task labels Aug 28, 2019
@philwebb philwebb changed the title Spring Boot CLI Exec Spring Boot CLI is unable to always capture ctrl-c Aug 28, 2019
wilkinsona pushed a commit that referenced this pull request Aug 29, 2019
Previously, when the Spring Boot CLI ran the Java command line for an
application it did a straight invocation which suppressed propagation of
signals to the JVM in certain circumstances (e.g within Docker containers).

This change prepends the command with exec which causes the Java process to
replace the script process in a shell ensuring that signals are propagated to
the Java process without suppression.

See gh-17976

Signed-off-by: Ben Hale <bhale@pivotal.io>
@wilkinsona wilkinsona modified the milestones: 2.1.x, 2.1.8 Aug 29, 2019
@wilkinsona
Copy link
Member

Thanks very much, @nebhale. The proposed change has been merged into 2.1.x and forwards into master.

pull bot pushed a commit to scope-demo/spring-boot that referenced this pull request Aug 29, 2019
Previously, when the Spring Boot CLI ran the Java command line for an
application it did a straight invocation which suppressed propagation of
signals to the JVM in certain circumstances (e.g within Docker containers).

This change prepends the command with exec which causes the Java process to
replace the script process in a shell ensuring that signals are propagated to
the Java process without suppression.

See spring-projectsgh-17976

Signed-off-by: Ben Hale <bhale@pivotal.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants