Re: [repro-devel] WebAdmin domain
Hello Scott,
Monday, December 17, 2007, 5:07:46 PM, you wrote:
>> I think that parameters table is a better place for admin password. We
>> can
>> move admin password from userStore to parameters table, if we found
>> admin
>> record in userStore, for backward compatibility
> [Scott] Why do you think it's best? I think the Users Tables make more
> sense. If in the future we want to have multiple administrators, and/or
> be able to flag regular users as administrators - then the Users tables
> gives more flexibility.
>
1) Users and admins - different things and must be managed
separatelly.
2) Users is proxy level and admins is application level. Any derived
work will introduce own management tool and own security for
management tool.
3) If we will need multiple admins we can/must create new db table.
>> > 7. #4 (from email below) is still a backwards compatibility issue
>> that
>> > needs to be solved before merging with mainline.
>>
>> May be we need silently move database to new location? If no db files
>> present in new location and they are exist in exe directory.
> [Scott] Some people (including myself) - are possibly running multiple
> copies of repro on the same box. This change will break that. How
> about: if not db directory is specified and db exists in working
> directory then use it - else use/create db from profile directory.
>
I think that using multiple repro instances is rare condition. If user
have knowledge how to run multiple servers, he can specify db dir in
command line. But if You insist, I will do it.
>> >> - It would be good if the parameters page showed the default
>> values
>> > -
>> >> perhaps we should load the parameters table with all of the compile
>> >> time
>> >> defaults at startup - thought?
>> >>
>>
>> We can not do this at this time. We need rewrite CommandLineParser for
>> expose default values, or create new class CommandLineDefault. One of
>> this classes will be seen by WebAdmin.
> [Scott] I think this is important to solve from a usability standpoint.
> We could move the default values into the ReproConfiguration class - The
> CommandLineParser class could use these defaults instead of the ones
> hardcoded in that class.
I think that adding class CommandLineDefault is best way.
>> 5) Remove Using.html from distro and add link to web page. Add
>> link to http://localhost:5080.
> Note: Link to localhost:5080 will only work if web admin is enabled and
> web port is set to 5080.
If user can change webadmin port, he can change link file too. :-)
--
Best regards,
Ruslan mailto:prostoruslan@xxxxxxxxx