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.

Sign up for GitHub

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

Jump to bottom

Implement usersInfo command #1497

Closed
Tracked by #7
b1ron opened this issue Nov 17, 2022 · 0 comments · Fixed by #3897
Closed
Tracked by #7

Implement usersInfo command #1497

b1ron opened this issue Nov 17, 2022 · 0 comments · Fixed by #3897
Assignees
@henvic
Labels
area/auth Issues about authentication and authorization code/feature Some user-visible feature is not implemented yet
Milestone
v1.18.0

Comments

@b1ron
Copy link
Contributor

b1ron commented Nov 17, 2022 •
edited by AlekSi
Loading

Blocked by #1491 and #1493.

What should be done?

With createUser and dropUser implemented, we should implement the basic usersInfo command.

We should accept the following usersInfo arguments:

  • usersInfo;
  • showCredentials – ignore for now;
  • showCustomData – ignore;
  • showPrivileges – ignore;
  • showAuthenticationRestrictions – ignore;
  • filter;
  • comment.

Tests should use createUser and dropUser commands, not insert and delete.

Where?

https://github.com/FerretDB/FerretDB/tree/main/internal/handler

Definition of Done

  • handler updated;
  • integration/compatibility tests added/updated;
  • spot refactorings done;
  • basic hidden user documentation page created.
@b1ron b1ron added the code/feature Some user-visible feature is not implemented yet label Nov 17, 2022
@AlekSi AlekSi mentioned this issue Dec 23, 2022
Closed
@AlekSi AlekSi added the not ready Issues that are not ready to be worked on; PRs that should skip CI label May 1, 2023
@AlekSi AlekSi added this to the v2.0.0 milestone Oct 5, 2023
@AlekSi AlekSi mentioned this issue Nov 30, 2023
Closed
@AlekSi AlekSi removed the not ready Issues that are not ready to be worked on; PRs that should skip CI label Dec 5, 2023
@henvic henvic mentioned this issue Dec 21, 2023
Merged
9 tasks
henvic added a commit that referenced this issue Jan 4, 2024
@henvic @AlekSi @rumyantseva
Implement usersInfo command (#3897)
74d7272 
Closes #1497


Co-authored-by: Alexey Palazhchenko <alexey.palazhchenko@ferretdb.io>
Co-authored-by: Elena Grahovac <elena@grahovac.me>
@AlekSi AlekSi modified the milestones: v2.0.0, v1.18.0 Jan 8, 2024
@AlekSi AlekSi added the area/auth Issues about authentication and authorization label Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@henvic henvic

Labels
area/auth Issues about authentication and authorization code/feature Some user-visible feature is not implemented yet
Projects
Current Engineering
Archived in project
Milestone
v1.18.0
Development

Successfully merging a pull request may close this issue.

Implement usersInfo command henvic/FerretDB
3 participants
@AlekSi @henvic @b1ron