Commit Graph

120 Commits

Author SHA1 Message Date
Mohd Bilal
24b98e1900 cli: refactor internal/projectmetadata package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6435
GitOrigin-RevId: d48b0412cb8f7d252018861cbbe65123eddfc412
2022-11-02 12:10:25 +00:00
Mohd Bilal
27875e0260 cli: port internal/hasura/v1metadata package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6378
GitOrigin-RevId: 5551a12f19f492bd34de6bb56b44f33d9386b53d
2022-10-31 09:14:10 +00:00
Mohd Bilal
a8a37b246b cli: port internal/hausra/sourceops/mssql to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6369
GitOrigin-RevId: a1dee4ec7bf2e3ef2d5b457e47cc4e0d1a6f7c07
2022-10-31 08:40:44 +00:00
Mohd Bilal
5a13c271f8 cli: port internal/hausra/sourceops/citus to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6367
GitOrigin-RevId: c71fdcf8c0422df899ca5054ae69eeedc157ac0d
2022-10-31 07:45:06 +00:00
Aravind K P
22ec874069 cli: port statestore/settings to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6492
Co-authored-by: Mohd Bilal <24944223+m-Bilal@users.noreply.github.com>
GitOrigin-RevId: 409b220908d773cc6b45bc7ebe11a6ea950e9d35
2022-10-28 07:51:58 +00:00
Aravind K P
e73a8f74e4 cli: port metadataobject/inherited_roles to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6406
GitOrigin-RevId: 4dcf132634578a8defd3678e11c03fce94bfb827
2022-10-27 09:51:42 +00:00
Aravind K P
ca6605a036 cli: port metadataobject/apilimits to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6397
Co-authored-by: Mohd Bilal <24944223+m-Bilal@users.noreply.github.com>
GitOrigin-RevId: ff266638f767da75da584f366d1a464ba6b70f0c
2022-10-27 08:28:23 +00:00
Aravind K P
a19a0c4b30 cli: port metadataobject/actions to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6425
GitOrigin-RevId: 87cb9f0244874065ed862f73eef93cfe73b82852
2022-10-27 07:49:44 +00:00
Aravind K P
7669b01e3a cli: port metadataobject/remoteschemas to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6413
GitOrigin-RevId: 3ffdf4076ba454bbbfe09cc61e401a64488637a8
2022-10-27 07:07:37 +00:00
Aravind K P
5217c5f0c1 cli: port metadataobject/network to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6408
GitOrigin-RevId: e3fb8e2b2bcef32b072818ac4c4a32124d7876d4
2022-10-27 06:18:50 +00:00
Mohd Bilal
edc40ea7be cli: refactor internal/scripts package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6458
GitOrigin-RevId: 579a7cba6780feee81f9ee06cab6ccb5f7a4cae3
2022-10-26 07:30:25 +00:00
Aravind K P
5d759189d0 cli: port metadataobject/query_collections to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6410
GitOrigin-RevId: d82eb90669f3a915f2a2fc9d87347ff64f4ee97b
2022-10-24 03:22:47 +00:00
Aravind K P
1aa5cbb195 cli: port metadataobject/allowlist package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6396
GitOrigin-RevId: 180eaa60613985334681bd9011f910c3f3d6afb2
2022-10-21 16:57:31 +00:00
Samir Talwar
c3afa0fdd7 Install and use ODBC Driver 18 for SQL Server (msodbcsql18).
This installs the ODBC Driver 18 for SQL Server in all our shipped Docker images, and update our tests and documentation accordingly.

This version supports arm64, and therefore can run natively (or via Docker) on macOS on aarch64.

`msodbcsql17` is still installed in production-targeted Docker images so that users do not _have_ to migrate to the new driver.

Nix expressions are packaged for the new driver, as it is not yet available in nixpkgs.

In this version, [the default encryption setting was changed from "no" to "yes"](https://techcommunity.microsoft.com/t5/sql-server-blog/odbc-driver-18-0-for-sql-server-released/ba-p/3169228). In addition, "mandatory" and "optional" were added as synonyms for "yes" and "no" respectively.

I have therefore modified all connection strings in tests to specify `Encrypt=optional` (and changed some from `Encrypt=no`). I chose "optional" rather than "no" because I feel it's more honest; these connection strings will work with or without an encrypted connection.

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6241
GitOrigin-RevId: 959f88dd1f271ef06a3616bc46b358f364f6cdfd
2022-10-21 16:25:04 +00:00
Aravind K P
3cec172f6f cli: port metadataobject/rest_endpoints to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6424
GitOrigin-RevId: 3b210e6f6b47756e7941fcb14e960674718855a1
2022-10-21 14:51:38 +00:00
Mohd Bilal
0c8b80ee71 cli: port commonmetadata package to use internal/errors package
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6318
GitOrigin-RevId: 2274c535f5475d50fbfdf645572b8553625a87fc
2022-10-20 15:18:46 +00:00
Aravind K P
754fb6e8b9 cli: port metadataobject/crontriggers to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6398
GitOrigin-RevId: 25f9964b3eecb2f89d482401d425752bd7d33708
2022-10-20 07:54:31 +00:00
Aravind K P
3d7effbcc1 cli: refactor metadataobject.WriteDiff to use std error
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6389
GitOrigin-RevId: 731b74ae5bb1ad0479c141f1e66795d9fd21e9e3
2022-10-20 06:58:52 +00:00
Aravind K P
e89895b5f1 cli: refactor metadataobject.GetFiles to use std error
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6388
GitOrigin-RevId: 4ae565fc7d7a9275473a736aac78a2da7d031260
2022-10-20 04:36:56 +00:00
Aravind K P
4b1ab8bf67 cli: refactor metadataobject.Object interface to use std error
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6387
GitOrigin-RevId: 5a18fb9b26eaf9bc9a1b55ae79ce18e4147f4aa3
2022-10-19 12:52:46 +00:00
Mohd Bilal
51424c0857 cli: use github.com/stretchr/testify/require assertions in internal/hasura/pgdump tests
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6364
GitOrigin-RevId: c6c99c092e6992d3905e1a1f58ea0373911238f8
2022-10-18 03:24:17 +00:00
Mohd Bilal
aa70c62bdd cli: port internal/hasura/v2query package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6379
GitOrigin-RevId: 787e5cce03006b0d248a79797923efb8fe6fa7b4
2022-10-17 18:28:56 +00:00
Mohd Bilal
081b9163c4 cli: port internal/hausra/sourceops/postgres to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6374
GitOrigin-RevId: 8b52f7409799d7ac3b43937cc66837ab3861cd03
2022-10-17 17:56:02 +00:00
Mohd Bilal
7044bf194d cli: port internal/hasura/v1graphql package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6322
GitOrigin-RevId: 1f1c8f13bd688881c48f3381dc9e3e7bacf5fc0c
2022-10-17 17:02:00 +00:00
Mohd Bilal
06138634bc cli: use github.com/stretchr/testify/require assertions in internal/hasura/catalogstate tests
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6360
GitOrigin-RevId: 1c0f1666fb0ca70d7cf00e6584c549f1bf044c69
2022-10-17 09:57:59 +00:00
Aravind K P
2e0cd9267b cli: port v1query package to use internal/errors
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6334
GitOrigin-RevId: 094745ed3c8e6aaab5f7f01236ddbebba0f043ba
2022-10-13 15:06:56 +00:00
Mohd Bilal
7b5d61d04f CLI: port catalogstate package to use new errors package
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6259
GitOrigin-RevId: be90386154d7906a41c2334db5cc24fe5febf774
2022-10-12 09:45:51 +00:00
Mohd Bilal
0bd013653a cli: port pgdump package to use new errors package
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6263
GitOrigin-RevId: e410548ecba70e556e41a0377952555c3b40c8ef
2022-10-12 08:47:21 +00:00
Mohd Bilal
56a26a74ee CLI: Add utility Match and print function for the new errors package
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6258
GitOrigin-RevId: fa3148c467b527fbe62cebca3177a39f9b02b38c
2022-10-11 16:41:57 +00:00
Aravind K P
a949af5e14 cli: refactor httpc package to use error framework
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6227
GitOrigin-RevId: fcf9ee64345309ddcba95e4369a6564defa54fc0
2022-10-11 13:40:46 +00:00
Mohd Bilal
c28ed6cc6c cli: Fix query collection export missing definition object and queries array in yaml
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/5954
GitOrigin-RevId: b7b29c43ca9c3b2934ea12e71bca2dadecc4c34b
2022-09-27 13:46:42 +00:00
Mohd Bilal
8d18a55212 cli: move AddDatabaseToHasura to testutil package
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/6069
GitOrigin-RevId: f7a3b98685c16fec613719eaa05024cd3c9cd3d4
2022-09-27 11:16:22 +00:00
Aravind K P
451f252c0f cli: [rfc] improve error handling
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4817
GitOrigin-RevId: 340f8d0cb3df496c1c0df1cc60fe3752dd58c75c
2022-09-21 08:29:37 +00:00
hasura-bot
df5e887098 cli: fix bug when exporting metadata to remote_schemas.yaml
GITHUB_PR_NUMBER: 8921
GITHUB_PR_URL: https://github.com/hasura/graphql-engine/pull/8921

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/5744
Co-authored-by: Ryohei KOMATSUYAMA <54347899+kmtym1998@users.noreply.github.com>
GitOrigin-RevId: cc0948b62c3caccfb2c814433bb1a84508e3c37e
2022-09-19 12:20:26 +00:00
Mohd Bilal
6a90332ab1 cli: Add --disallow-inconsistent-metadata flag to hasura metadata apply command
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/5045
GitOrigin-RevId: 1e8da16285705e3e439f1e3e3e4c495c715a902b
2022-08-16 09:59:03 +00:00
Mohd Bilal
48a0a34a92 cli: add support for health_check metadata object in sources
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/5287
GitOrigin-RevId: be7e1d19b9875cf05a52a9894cf2bda58b209733
2022-08-03 13:17:36 +00:00
hasura-bot
92f601587c cli: use 2-spaces indent for graphql query in metadata (fix #8469)
GITHUB_PR_NUMBER: 8519
GITHUB_PR_URL: https://github.com/hasura/graphql-engine/pull/8519

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4544
Co-authored-by: Yudai Shinnoki <2777896+shinnoki@users.noreply.github.com>
Co-authored-by: Divi <32202683+imperfect-fourth@users.noreply.github.com>
Co-authored-by: Aravind K P <8335904+scriptonist@users.noreply.github.com>
GitOrigin-RevId: 3d9f9d1bf940e528a1ad9fab67b6ae818927ffac
2022-06-27 05:27:19 +00:00
Divi
c668aa1158 cli: improve error reporting in metadata apply command
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4454
Co-authored-by: Aravind K P <8335904+scriptonist@users.noreply.github.com>
GitOrigin-RevId: ae66bdfdc140c1a2a7747fb0ff2acd6f7b577d3f
2022-06-15 15:45:38 +00:00
Divi
b50aecdae4 cli: fix perfomance regression with large metadata in metadata apply
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4661
Co-authored-by: Aravind K P <8335904+scriptonist@users.noreply.github.com>
GitOrigin-RevId: 48050e899592878c47c382281605a0c8666ddc8b
2022-06-15 12:39:43 +00:00
Divi
b7c414a875 ci: add action to release cli-ext binaries
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4342
Co-authored-by: Vishnu Bharathi <4211715+scriptnull@users.noreply.github.com>
GitOrigin-RevId: 2b7686bda333d977aa20a1ff10f7b3179509eac3
2022-06-07 22:25:41 +00:00
Aravind K P
23f32c3db4 cli: fix performance regression in v2.5.0
closes https://github.com/hasura/graphql-engine/issues/8398

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4313
GitOrigin-RevId: 3f24434b8256ec30838ce0014a04f8dfdca7f0d5
2022-04-21 11:55:12 +00:00
Aravind K P
ff255bd065 cli: support remote relationships metadata
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/4274
GitOrigin-RevId: 5a839a39cc47268e8da52d039b91fbc35fec8b13
2022-04-18 11:07:46 +00:00
Aravind K P
e46777b7ea cli: fix formatting errors in query collection metadata
closes https://github.com/hasura/graphql-engine/issues/7616
closes https://github.com/hasura/graphql-engine/issues/8205

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3955
GitOrigin-RevId: dc97e23e78d522cb7ac4d57ad825526eaa90a4de
2022-03-31 14:02:49 +00:00
Aravind K P
649ef41e3c cli: fix remote schema formatting errors in metadata
closes https://github.com/hasura/graphql-engine/issues/7608
closes https://github.com/hasura/graphql-engine/issues/7459

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3951
GitOrigin-RevId: bad3505c79fab10453580a6a43ad7e71bc2d3753
2022-03-31 08:28:30 +00:00
Vishnu Bharathi
2e04bd5ac2 cli: add support for customization field in sources metadata
closes https://github.com/hasura/graphql-engine/issues/8292

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3954
GitOrigin-RevId: cc27dcffbb71579136a3cae7d99ecd239eb37ed2
2022-03-10 16:47:03 +00:00
Aravind K P
2ff0f25e08 cli: refactor metadataobject package
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3943
GitOrigin-RevId: 4cc8f67dd8b6ba0e78f1c35d415f89bd1b901705
2022-03-10 08:13:50 +00:00
paritosh-08
a0c83b8e99 server: add inconsistency information in reload_metadata API call
PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3860
GitOrigin-RevId: 02d1963946d69fdf6886aa61806eb71db60f8931
2022-03-03 13:34:44 +00:00
Divi
d87b961378 cli: fix metadata version being set to 3 for v2 project
closes https://github.com/hasura/graphql-engine/issues/8148

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3619
GitOrigin-RevId: d10b715fc71dcc89bb0874f1c115ad9e562cfd6c
2022-02-23 10:53:22 +00:00
Kali Vara Purushotham Santhati
780d942fad cli: skip tls verfication for all API requests when insecure-skip-tls-verify flag is set
closes: https://github.com/hasura/graphql-engine/issues/4926

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/3307
Co-authored-by: Aravind K P <8335904+scriptonist@users.noreply.github.com>
GitOrigin-RevId: 90222fc2e5b176fbf5721fea3f5b2af929ca93f4
2022-02-04 11:11:37 +00:00
Antoine Leblanc
d66d2d81fd server: use placeholder to prevent empty query roots
### Description

The GraphQL spec has to conflicting requirements:
1. an object must contain at least one field: the schema may not contain empty objects
2. the _query_root_ must always be present

Given _1_, the schema generation code removes from the schema all fields that would result in empty objects, such as a table for which a user does not have select permissions. But, as a result, our code also potentially removes _query_root_ if it is empty, breaking _2_.

This PR introduces a dummy "placeholder" field in the query root if it's empty, to ensure we never remove it from the schema.

### Remaining work

- [x] changelog entry
- [x] tests

PR-URL: https://github.com/hasura/graphql-engine-mono/pull/148
GitOrigin-RevId: bfd6bfcc2f3de92900b6ba566012f093399ca037
2022-01-27 18:08:11 +00:00