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

Support missing or empty projection clause for commands #158

Closed
amorton opened this issue Feb 20, 2023 · 0 comments · Fixed by #286
Closed

Support missing or empty projection clause for commands #158

amorton opened this issue Feb 20, 2023 · 0 comments · Fixed by #286
Assignees
@tatu-at-datastax

Comments

@amorton
Copy link
Contributor

amorton commented Feb 20, 2023 •
edited by sync-by-unito bot
Loading

All commands that support a projection clause must support the following:

  • projection field not present in the command JSON object
  • projection field present, but an empty JSON object.

If the projection field is present, but is not a JSON object (e.g. {"projection" : "yes please :)"}) the service should return an error.

See this usecase form the client stargate/stargate-mongoose#18 (reply in thread)
This was referenced Mar 22, 2023
Closed
Merged
tatu-at-datastax added a commit that referenced this issue Mar 24, 2023
@tatu-at-datastax
Fix #158: support empty Projection clause; add full decoding of proje…
7870162 
…ction definition (but not yet processing) (#286)
This was referenced Mar 27, 2023
Closed
Merged
tatu-at-datastax added a commit that referenced this issue Mar 27, 2023
@tatu-at-datastax
Fix remaining gap in #158, connect findOneAndUpdate() to projection (#…
328cc43 
…301)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tatu-at-datastax tatu-at-datastax

Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@amorton @tatu-at-datastax @maheshrajamani