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

fix(GraphQL): Unable to run queries with order as array #6310

Closed
wants to merge 158 commits into from
Closed

fix(GraphQL): Unable to run queries with order as array #6310

wants to merge 158 commits into from

Conversation

douglasmuraoka
Copy link
Contributor

An internal server error is thrown if the order query parameter is an array.
Currently, order only works with a single value.

greenkeeper bot and others added 30 commits September 13, 2019 15:51
@greenkeeper
Update graphql to the latest version 🚀 (parse-community#6048)
8b8eed2 
* fix(package): update graphql to version 14.5.5

* chore(package): update lockfile package-lock.json
@greenkeeper
Update eslint to the latest version 🚀 (parse-community#6050)
1c8e814 
* chore(package): update eslint to version 6.4.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update graphql to the latest version 🚀 (parse-community#6054)
fc57ff4 
* fix(package): update graphql to version 14.5.6

* chore(package): update lockfile package-lock.json
@davimacedo @douglasmuraoka
Update GraphQL readme section (parse-community#6056)
d704740
@greenkeeper
Update cross-env to the latest version 🚀 (parse-community#6058)
7225c8e 
* chore(package): update cross-env to version 6.0.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update flow-bin to the latest version 🚀 (parse-community#6063)
da4871a 
* chore(package): update flow-bin to version 0.108.0

* chore(package): update lockfile package-lock.json
@brodo @davimacedo
Use #!/bin/bash instead of #!/bin/sh (parse-community#6062)
431cb94 
The script uses double square brackets, which are a non-standard extension to `[]`. Some shells (e.g. dash, the default shell under Debian) do not support double square brackets. The shebang line should reflect that.
@greenkeeper
Update node-rsa to the latest version 🚀 (parse-community#6064)
065caab 
* fix(package): update node-rsa to version 1.0.6

* chore(package): update lockfile package-lock.json
@brodo @davimacedo
Install parse-server 3.9.0 insted of 2.2 (parse-community#6069)
24bed70 
The script currently writes a package.json with a dependency on parse-server version 2.2. This should probably always be automatically updated to the latest version using some CI magic.
@greenkeeper
Update graphql to the latest version 🚀 (parse-community#6070)
4a9a275 
* fix(package): update graphql to version 14.5.7

* chore(package): update lockfile package-lock.json
@greenkeeper
Update jasmine to the latest version 🚀 (parse-community#6071)
f56ac26 
* chore(package): update jasmine to version 3.5.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update lint-staged to the latest version 🚀 (parse-community#6074)
ffd7389 
* chore(package): update lint-staged to version 9.3.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update babel7 to the latest version 🚀 (parse-community#6078)
5b3deb8 
* chore(package): update @babel/cli to version 7.6.2

* chore(package): update @babel/core to version 7.6.2

* chore(package): update @babel/plugin-proposal-object-rest-spread to version 7.6.2

* chore(package): update @babel/preset-env to version 7.6.2

* chore(package): update lockfile package-lock.json
@greenkeeper
Update graphql to the latest version 🚀 (parse-community#6084)
eec739a 
* fix(package): update graphql to version 14.5.8

* chore(package): update lockfile package-lock.json
@greenkeeper
Update commander to the latest version 🚀 (parse-community#6085)
ebd1752 
* fix(package): update commander to version 3.0.2

* chore(package): update lockfile package-lock.json
@dependabot-preview
Bump apollo-server-express from 2.9.3 to 2.9.4 (parse-community#6083)
48e03c8 
Bumps [apollo-server-express](https://github.com/apollographql/apollo-server) from 2.9.3 to 2.9.4.
- [Release notes](https://github.com/apollographql/apollo-server/releases)
- [Changelog](https://github.com/apollographql/apollo-server/blob/master/CHANGELOG.md)
- [Commits](https://github.com/apollographql/apollo-server/compare/apollo-server-express@2.9.3...apollo-server-express@2.9.4)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@dependabot-preview
Bump lint-staged from 9.3.0 to 9.4.0 (parse-community#6086)
dd08786 
Bumps [lint-staged](https://github.com/okonet/lint-staged) from 9.3.0 to 9.4.0.
- [Release notes](https://github.com/okonet/lint-staged/releases)
- [Commits](lint-staged/lint-staged@v9.3.0...v9.4.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@alann-maulana @dplewis
Microsoft Graph Authentication (parse-community#6051)
38e0ff9 
* add microsoft graph auth

* change mail to id

* add graph user id and email

* add microsoft graph auth test case

* remove validating auth data using mail

* add test case to AuthenticationAdapters

* fix indentation

* fix httpsRequest and fakeClaim not found

* add newline eof last

* fix test in auth adapter

* fix unhandled promise rejection
@greenkeeper
Update pg-promise to the latest version 🚀 (parse-community#6076)
fe90582 
* fix(package): update pg-promise to version 9.2.0

* chore(package): update lockfile package-lock.json
@dependabot-preview
Bump pg-promise from 9.1.4 to 9.2.1 (parse-community#6077)
75a27ca 
Bumps [pg-promise](https://github.com/vitaly-t/pg-promise) from 9.1.4 to 9.2.1.
- [Release notes](https://github.com/vitaly-t/pg-promise/releases)
- [Commits](vitaly-t/pg-promise@9.1.4...9.2.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@greenkeeper
Update husky to the latest version 🚀 (parse-community#6090)
46550fe 
* chore(package): update husky to version 3.0.6

* chore(package): update lockfile package-lock.json
@greenkeeper
Update eslint to the latest version 🚀 (parse-community#6092)
6aaebb6 
* chore(package): update eslint to version 6.5.0

* chore(package): update lockfile package-lock.json
@dependabot-preview
Bump husky from 3.0.6 to 3.0.7 (parse-community#6093)
dbd5f73 
Bumps [husky](https://github.com/typicode/husky) from 3.0.6 to 3.0.7.
- [Release notes](https://github.com/typicode/husky/releases)
- [Changelog](https://github.com/typicode/husky/blob/master/CHANGELOG.md)
- [Commits](typicode/husky@v3.0.6...v3.0.7)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@greenkeeper
Update lint-staged to the latest version 🚀 (parse-community#6096)
8f53034 
* chore(package): update lint-staged to version 9.4.1

* chore(package): update lockfile package-lock.json
@dependabot-preview
Bump eslint from 6.5.0 to 6.5.1 (parse-community#6097)
c63864f 
Bumps [eslint](https://github.com/eslint/eslint) from 6.5.0 to 6.5.1.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md)
- [Commits](eslint/eslint@v6.5.0...v6.5.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
@greenkeeper
Update cross-env to the latest version 🚀 (parse-community#6098)
6a45213 
* chore(package): update cross-env to version 6.0.2

* chore(package): update lockfile package-lock.json
@acinader @davimacedo
Update the pgp public key block (parse-community#6099)
4998135
@greenkeeper
Update husky to the latest version 🚀 (parse-community#6100)
9cf3b52 
* chore(package): update husky to version 3.0.8

* chore(package): update lockfile package-lock.json
@Moumouls @davimacedo
GraphQL: ACL (parse-community#5957)
2290145 
* Spec

Fix Spec

* Add ACL Type + Input

* Improvements

* Fix
@dependabot-preview
Bump cross-env from 6.0.2 to 6.0.3 (parse-community#6105)
27616c9 
Bumps [cross-env](https://github.com/kentcdodds/cross-env) from 6.0.2 to 6.0.3.
- [Release notes](https://github.com/kentcdodds/cross-env/releases)
- [Changelog](https://github.com/kentcdodds/cross-env/blob/master/CHANGELOG.md)
- [Commits](kentcdodds/cross-env@v6.0.2...v6.0.3)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Moumouls and others added 23 commits December 11, 2019 21:47
@Moumouls @davimacedo
Handle required fields (parse-community#6271)
e0e06ef
@greenkeeper
Update semver to the latest version 🚀 (parse-community#6283)
21de2bd 
* fix(package): update semver to version 7.0.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update ws to the latest version 🚀 (parse-community#6281)
87e56cb 
* fix(package): update ws to version 7.2.1

* chore(package): update lockfile package-lock.json
@greenkeeper
Update apollo-client to the latest version 🚀 (parse-community#6282)
e29e228 
* chore(package): update apollo-cache-inmemory to version 1.6.5

* chore(package): update apollo-client to version 2.6.6

* chore(package): update apollo-utilities to version 1.3.3

* chore(package): update lockfile package-lock.json
@greenkeeper
Update flow-bin to the latest version 🚀 (parse-community#6280)
2fc328e 
* chore(package): update flow-bin to version 0.114.0

* chore(package): update lockfile package-lock.json
@Moumouls @davimacedo
GraphQL: Support required fields on output type (parse-community#6279)
a72ab50 
* Handle required fields

* Fix output fields
@vitaly-t @dplewis
Update PostgresStorageAdapter.js (parse-community#6275)
2d665c9 
* Update PostgresStorageAdapter.js

Improving use of the `await.async` notation in relation to `pg-promise`, and in general.

* Update PostgresStorageAdapter.js

* Update PostgresStorageAdapter.js

Correcting some results.

* Update PostgresStorageAdapter.js
@Klaitos @dplewis
Add missing encodeURIComponent on username password reset (parse-comm…
d9f36b0 
…unity#6278)

* Add missing encodeURIComponent on username

* Add new unit test on encoded username password reset redirect
@greenkeeper
Update semver to the latest version 🚀 (parse-community#6284)
7944ef1 
* fix(package): update semver to version 7.1.0

* chore(package): update lockfile package-lock.json
@rhuanbarreto @dplewis
parse-community#6101 Let users define objectId (parse-community#6177)
8bc201d 
* parse-community#6101 Let users define objectId

* Add `allowCustomObjectId` to PS Option

* Add checking in objectId creation

* Add test

* Update docs

* Update definition

* Change default to false

* throw on empty, null, undefined

* better tests

* unused async

* removed comment

* retain comment

* Linting fix according to contributing spec.
@greenkeeper
Update apollo-server to the latest version 🚀 (parse-community#6285)
1833909 
* fix(package): update apollo-server-express to version 2.9.14

* chore(package): update lockfile package-lock.json
@greenkeeper
Update semver to the latest version 🚀 (parse-community#6286)
71e7894 
* fix(package): update semver to version 7.1.1

* chore(package): update lockfile package-lock.json
@greenkeeper
Update mongodb to the latest version 🚀 (parse-community#6290)
a616fa0 
* fix(package): update mongodb to version 3.4.1

* chore(package): update lockfile package-lock.json
@greenkeeper
Update eslint to the latest version 🚀 (parse-community#6291)
81ca1fc 
* chore(package): update eslint to version 6.8.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update babel7 to the latest version 🚀 (parse-community#6289)
b0fcb93 
* chore(package): update @babel/cli to version 7.7.7

* chore(package): update @babel/core to version 7.7.7

* chore(package): update @babel/plugin-proposal-object-rest-spread to version 7.7.7

* chore(package): update @babel/preset-env to version 7.7.7

* chore(package): update lockfile package-lock.json
@greenkeeper
Update nyc to the latest version 🚀 (parse-community#6294)
49a0405 
* chore(package): update nyc to version 15.0.0

* chore(package): update lockfile package-lock.json
@dplewis
Add maxLogFiles option (parse-community#6296)
1a99cb3 
https://community.parseplatform.org/t/server-log-retention/867/2

Maximum number of logs to keep. If not set, no logs will be removed. This can be a number of files or number of days. If using days, add 'd' as the suffix. (default: null).

I'll run this in production for a few days. I assume it works.

Useful for saving disk space.

Update winston-daily-rotate-file package to 4.4.0
@greenkeeper
Update parse to the latest version 🚀 (parse-community#6297)
ef4248c 
* fix(package): update parse to version 2.11.0

* chore(package): update lockfile package-lock.json
@greenkeeper
Update pg-promise to the latest version 🚀 (parse-community#6298)
47454ef 
* fix(package): update pg-promise to version 10.3.2

* chore(package): update lockfile package-lock.json
@JeromeDeLeon @dplewis
Add syntax highlighting for bash codes (parse-community#6302)
832a081
@greenkeeper
Update apollo-server to the latest version 🚀 (parse-community#6303)
558ce66 
* fix(package): update apollo-server-express to version 2.9.15

* chore(package): update lockfile package-lock.json
@douglasmuraoka @dplewis
fix(GraphQL): Timeout when fetching huge collections (parse-community…
d7236ba 
…#6304)

* fix(GraphQL): Timeout when fetching huge collections

Currently, when not specifying a `limit` to the GraphQL find-like query, it tries to fetch the entire collection of objects from a class. However, if the class contains a huge set of objects, it is never resolved and results in timeout.

In order to solve this kind of problem, `parse-server` allows us to define a `maxLimit` parameter when initialized, which limits the maximum number of objects fetched per query; but it is not properly considered when the `limit` is undefined.

* fix: Keep same behavior as REST fetch
@douglasmuraoka
fix(GraphQL): Unable to run queries with order as array
851c57f 
An internal server error is thrown if the `order` query parameter is an array.
Currently, `order` only works with a single value.
@dplewis
Copy link
Member

dplewis commented Jan 2, 2020

Can you provide a test case?

@davimacedo davimacedo closed this Jan 6, 2020
@davimacedo davimacedo mentioned this pull request Jan 6, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.