Bug 46436

Summary:	Improve error reporting in Proxy Gui
Product:	JMeter - Now in Github	Reporter:	Rose-Line Werner <wernerr>
Component:	Main	Assignee:	JMeter issues mailing list <issues>
Status:	RESOLVED FIXED
Severity:	enhancement	CC:	hauser, milamber
Priority:	P2
Version:	Nightly (Please specify date)
Target Milestone:	---
Hardware:	PC
OS:	Linux
Attachments:	patch_bug46436.txt Patch to add French translation for the two new messages added by the main patch.

Description Rose-Line Werner 2008-12-22 08:14:51 UTC

Hi,

I'm using JMeter r723926 (Version of 6 Dec 08). 

When I set up the JMeter Http Proxy Controller with a port which is already in use by another process then, although a warning is posted in the jmeter.log, there is no alert in the Gui (Http Proxy Server Controller) notifying the user about the occupied port.

Especially for jMeter beginners which are not used to check the log file, an alert like the following would be very helpful:

When pressing the START button of the Http Proxy Server Controller, the alert "Port XYZ is already in use, please choose a different one." is displayed.

Thanks for your great work.

Comment 1 Rose-Line Werner 2009-01-14 07:28:37 UTC

Created attachment 23118 [details]
patch_bug46436.txt

with the patch, an alert window will be shown if the proxy port is already in use. Additionally, an error log entry is written to jmeter.log

Comment 2 Sebb 2009-01-24 10:24:56 UTC

Thanks for the patch.

Although the patch worked, it was still a bit confusing for the user as the Stop/Start buttons were still updated as if the proxy had started correctly.

So I ended up making a more substantial change; see SVN revisisions:

r737409
r737410
r737411

Comment 3 Milamber 2009-01-25 01:31:57 UTC

Created attachment 23175 [details]
Patch to add French translation for the two new messages added by the main patch.

Comment 4 Ralf Hauser 2009-01-25 02:39:54 UTC

see also Bug 46438

Comment 5 Sebb 2009-01-25 10:34:33 UTC

Thanks, applied patch to French properties file

Comment 6 The ASF infrastructure team 2022-09-24 20:37:42 UTC

This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/2186