modules instance name restrictions

[Alan DeKok]

Thibault Le Meur wrote:
> * I defined my own pap module with the name 'pap-myorg' and expected it
> to set Auth-Type to PAP-MYORG, but in fact it wasn't setting the
> Auth-Type at all (moreover I saw no notice in the log about not beeing
> able to set the Auth-Type).
> * I changed the instance name to 'papmyorg' and now the module sets the
> Auth-Type to PAPMYORG.

  What what does debug mode say?

> I tried to find in the doc where the restrictions on module instances
> names were defined, but didn't found any reference to this. And given
> the fact that some standard modules have specific chars such as '_' or
> '-', I thought there was little constraints.

  There are no constraints on using "-".  When I test it with
"pap-myorg", it works.

> Is there any module instance naming convention written somewhere in the
> provided documentation (or online)?
> Maybe be it could be interresting to have a warning in the radius debug
> log in order to notice the administrator that Auth-Type wasn't set due
> to module naming restrictions?

  The debug log says what it's doing any why.  It doesn't say what is
NOT happening.

  Alan DeKok.