`.enabled()` logic incorrectly returns true when namespace contains asterisk

debug.disable('*');
debug.enable('foo');
assert.not(debug.enabled('bar'));
assert.not(debug.enabled('bar*')); // throws; returns true

Has to be solved in two ways:

• More stringent checks on namespaces (as they shouldn’t contain asterisks)
• Remove the check for ns[ns.length-1]==='*', which makes absolutely no sense

[jasonk]

According to the documentation, this is by design:

If you append a “*” to the end of your name, it will always be enabled regardless of the setting of the DEBUG environment variable.

[stephenhandley]

naively seems like if (name === '*') { return true; } would be less surprising.

would be interesting to know if anyone made use of the bar* thing in some way