Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 48872

Summary:	[Wizards] No validation on module name in sharing wizard
Product:	[Eclipse Project] Platform	Reporter:	Jean-Michel Lemieux <jean-michel_lemieux>
Component:	CVS	Assignee:	platform-cvs-inbox <platform-cvs-inbox>
Status:	CLOSED WONTFIX	QA Contact:
Severity:	normal
Priority:	P5	CC:	krzysztof.daniel, Michael.Valenta, tomasz.zarna
Version:	3.0	Keywords:	helpwanted
Target Milestone:	---
Hardware:	PC
OS:	Windows XP
Whiteboard:	stalebug

Description Jean-Michel Lemieux

2003-12-16 11:36:32 EST

I20031216
You can enter invalid strings in the module field without validation. For
example, a/n///////////. 

This actually ends up in a module shared with a/n///////. We should at least
strip the extra '/'s and validate in the wizard.

Comment 1 Jean-Michel Lemieux

2004-06-11 16:51:31 EDT

Post 3.0

Comment 2 Krzysztof Daniel

2007-08-09 10:19:54 EDT

I was not able to find module pattern, but I propose to exclude following characters:

*/|:\

Comment 3 Michael Valenta

2007-08-09 16:22:23 EDT

You need to keep a few things in mind when validating a module name. First of all, it is now a path so the slash (/) character is not invalid. However, two or more in a row are. Also, the colon character (:) is valid on some platforms (e.g. Linux).

Comment 4 Tomasz Zarna

2008-04-10 11:06:09 EDT

The case described in comment 0 works for me when tried on I20080409-1425, it looks like the stripping has been added in the meantime. 

There is still no validation in the wizard though. Moreover, there might be other cases mentioned by Christopher (comment 2) and Michael (comment 3), so let's keep the bug opened.

Comment 5 Eclipse Webmaster

2019-09-06 16:11:49 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet.

If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

Comment 6 Eclipse Genie

2021-10-27 18:05:20 EDT

This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. As such, we're closing this bug.

If you have further information on the current state of the bug, please add it and reopen this bug. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant.

--
The automated Eclipse Genie.