I just fixed a bug in my admin command that caused it to fail when
multiple clients were using the same command. I forgot to add
@Scoped(PerLookup.class). Without that, a second client executing
the same command that a first client is executing will overwrite
the parameters from the first client.
Seems to me *all* admin commands should have @Scoped(PerLookup.class),
yet I see many others that are missing it as well. (I suppose an
exception could be made for commands with no parameters and no
writable fields, but is that an important use case? I doubt it.)
Anybody disagree?
Isn't this a bug just waiting to happen?