Bug 48872 - [Wizards] No validation on module name in sharing wizard
Summary: [Wizards] No validation on module name in sharing wizard
Status: CLOSED WONTFIX
Alias: None
Product: Platform
Classification: Eclipse Project
Component: CVS (show other bugs)
Version: 3.0   Edit
Hardware: PC Windows XP
: P5 normal (vote)
Target Milestone: ---   Edit
Assignee: platform-cvs-inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords: helpwanted
Depends on:
Blocks:
 
Reported: 2003-12-16 11:36 EST by Jean-Michel Lemieux CLA
Modified: 2021-10-27 18:05 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jean-Michel Lemieux CLA 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 CLA 2004-06-11 16:51:31 EDT
Post 3.0
Comment 2 Krzysztof Daniel CLA 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 CLA 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 CLA 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 CLA 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 CLA 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.