Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create a publisher mode that does not allow any new user registrations #356

Closed
wants to merge 1 commit into from

Conversation

JoshData
Copy link
Contributor

@JoshData JoshData commented Feb 5, 2013

This patch adds a new option ckan.auth.create_user which when set to true prevents all new user registrations (via site and API). It is false by default.

This is useful in a setup where the site should take no unauthorized user input.

We can't deploy CKAN 2 without this at HealthData.gov.

…se prevents all new user registrations (via site and API), which is useful in a locked-down publisher mode
@ghost ghost assigned seanh Feb 12, 2013
seanh pushed a commit that referenced this pull request Feb 14, 2013
This makes the logic much easier to see I think
@seanh
Copy link
Contributor

seanh commented Feb 14, 2013

@tauberer I want to add some commits to this so I've created a new pull request here: #399

@seanh seanh closed this Feb 14, 2013
seanh pushed a commit that referenced this pull request Feb 14, 2013
If ckan.auth.create_user = False then don't let anyone create users,
either via the API or via the web interface.
seanh pushed a commit that referenced this pull request Feb 14, 2013
If not allowed to create users via api then say 'not authorized to
create user via api', if not allowed to create users at all then say
'not authorized to create users'.
nigelbabu added a commit that referenced this pull request Mar 26, 2013
nigelbabu added a commit that referenced this pull request Mar 26, 2013
* Remove the old commented functional tests testing against the
  templates.
* New tests that test against the API.
nigelbabu added a commit that referenced this pull request Jun 18, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants