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

Sync Function requireAdmin() functionality #3276

Closed
JFlath opened this Issue Feb 6, 2018 · 0 comments

Comments

Projects
None yet
3 participants
@JFlath
Copy link
Collaborator

JFlath commented Feb 6, 2018

Following on from a forum issue here:
https://forums.couchbase.com/t/sync-gateway-check-if-admin-call-superuser/15658/3

Whilst we do have a way of implementing this functionality in a Sync Function as described, it would be nice to have this neatly in it's own function. I did consider if it would be nicer to do something like requireUser("ADMIN") or similar, but we can't really reclaim the namespace there - a new function would be nicer.

@JFlath JFlath self-assigned this Feb 6, 2018

JFlath added a commit that referenced this issue Feb 6, 2018

@hideki hideki added the review label Feb 6, 2018

JFlath added a commit that referenced this issue Feb 6, 2018

JFlath added a commit that referenced this issue Feb 6, 2018

JFlath added a commit that referenced this issue Feb 6, 2018

@adamcfraser adamcfraser added this to the 2.1.0 milestone Feb 20, 2018

JFlath added a commit that referenced this issue Apr 3, 2018

JFlath added a commit that referenced this issue Apr 12, 2018

adamcfraser added a commit that referenced this issue Apr 12, 2018

@adamcfraser adamcfraser removed the review label Apr 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment