[
https://issues.apache.org/jira/browse/WAVE-313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13181056#comment-13181056
]
[email protected] commented on WAVE-313:
----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3296/#review4221
-----------------------------------------------------------
/src/org/waveprotocol/box/server/rpc/UserRegistrationServlet.java
<https://reviews.apache.org/r/3296/#comment9519>
Rename in this class to registrationDisabled for better readability.
/src/org/waveprotocol/box/server/rpc/UserRegistrationServlet.java
<https://reviews.apache.org/r/3296/#comment9520>
While you are here can you move this line into the else statement where the
registration actually completes. It makes the intention much clearer if you'd
ask me.
- Lennard
On 2012-01-04 15:37:37, Ali Lown wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/3296/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2012-01-04 15:37:37)
bq.
bq.
bq. Review request for wave.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. Implements suggestions in WAVE-313:
bq. - Adds a hideRegistration option to the server config file
bq. - Removes the auth/registration route when this is the case
bq. - Adds a RegistrationRobot to let the admin user always create new users
bq.
bq. Know limitations/features:
bq. - Doesn't remove any 'register' links - server 404s when they are clicked.
bq. - Manually created users don't get WelcomeBot'd
bq.
bq.
bq. This addresses bug WAVE-313.
bq. https://issues.apache.org/jira/browse/WAVE-313
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. /server-config.xml 1213039
bq. /server.config.example 1213039
bq. /src/org/waveprotocol/box/server/CoreSettings.java 1213039
bq. /src/org/waveprotocol/box/server/ServerMain.java 1213039
bq. /src/org/waveprotocol/box/server/gxp/UserRegistrationPage.gxp 1213039
bq. /src/org/waveprotocol/box/server/robots/agent/RobotAgentUtil.java
1213039
bq.
/src/org/waveprotocol/box/server/robots/agent/registration/RegistrationRobot.java
PRE-CREATION
bq. /src/org/waveprotocol/box/server/rpc/UserRegistrationServlet.java
1213039
bq. /test/org/waveprotocol/box/server/rpc/UserRegistrationServletTest.java
1213039
bq.
bq. Diff: https://reviews.apache.org/r/3296/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Compiled + run. Tried to access auth/register with hideRegistration=false
and hideRegistration=true.
bq. Logged in as the admin user and tried to create new accounts with the
RegistrationRobot - then logged into new accounts to check valid.
bq.
bq.
bq. Thanks,
bq.
bq. Ali
bq.
bq.
> option to disable "register new user" on web frontend, possibility for
> admin-user to add manually new users
> -----------------------------------------------------------------------------------------------------------
>
> Key: WAVE-313
> URL: https://issues.apache.org/jira/browse/WAVE-313
> Project: Wave
> Issue Type: New Feature
> Components: Web Client
> Reporter: JGB
> Assignee: Ali Lown
> Priority: Minor
> Labels: features
>
> As today, when you set up a new Wave-in-a-box-Server, everyone who has access
> to the web Client can create a new user. I think, there are a lot of people
> who would like to create private Wave-Server, so it would be nice to have an
> option to disable the "register new users"-site. As a consequence of this,
> there should also be the possibility to register as admin-user manually new
> accounts.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
