Skip to content

Commit 7a1751d

Browse files
improve PidPoller
1 parent cb493f9 commit 7a1751d

1 file changed

Lines changed: 6 additions & 8 deletions

File tree

src/main/java/com/testingbot/tunnel/PidPoller.java

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -32,22 +32,20 @@ public PidPoller(App app) {
3232
@Override
3333
public void run() {
3434
File f = new File(pidFileName);
35-
if (f.exists()) {
36-
f.delete();
35+
if (f.exists() && !f.delete()) {
36+
Logger.getLogger(PidPoller.class.getName()).log(Level.WARNING, "Could not delete pid file: {0}", pidFileName);
3737
}
3838
}
3939
};
4040

4141
pidFile = new File(fileName);
4242
if (!pidFile.exists()) {
43-
try {
44-
FileWriter fw = new FileWriter(pidFile.getAbsoluteFile());
45-
try (BufferedWriter bw = new BufferedWriter(fw)) {
46-
bw.write("TestingBot Tunnel, Remove this file to shutdown the tunnel");
47-
}
43+
try (FileWriter fw = new FileWriter(pidFile.getAbsoluteFile());
44+
BufferedWriter bw = new BufferedWriter(fw)) {
45+
bw.write("TestingBot Tunnel, Remove this file to shutdown the tunnel");
4846
Logger.getLogger(PidPoller.class.getName()).log(Level.INFO, "Pid file: {0}", pidFile.getAbsoluteFile().toString());
4947
} catch (IOException ex) {
50-
Logger.getLogger(PidPoller.class.getName()).log(Level.SEVERE, "Can't create testingbot pidfile in current directory");
48+
Logger.getLogger(PidPoller.class.getName()).log(Level.SEVERE, "Can't create testingbot pidfile in current directory", ex);
5149
return;
5250
}
5351
}

0 commit comments

Comments
 (0)