SeaQL / seaography

🧭 GraphQL framework for SeaORM
Apache License 2.0
393 stars 37 forks source link

Postgres example Broken with reached the recursion limit error #106

Closed aadi58002 closed 1 year ago

aadi58002 commented 1 year ago

Description

Steps to Reproduce

  1. Setup a postgres database with docker or podman.
  2. Apply the sakil-schema.sql and sakila-data.sql
  3. Get the Ip of the container running postgres database with docker inspect -f '{{range.NetworkSettings.Networks}}{{.IPAddress}}{{end}}' <postgres_container>
  4. export the DATABASE_URL with the postgres IP (for me it is 10.89.1.8)-> export DATABASE_URL="postgres://postgres:postgres@10.89.1.8/sakila"
  5. Run cargo run on the postgres example

Expected Behavior

  1. Getting the access to a graphql interactive console on port 8000. To Test out seaography

Actual Behavior

  1. cargo run fails with the following message Screenshot_20221205_094030

Reproduces How Often

  1. Always

Versions

├── sea-orm v0.10.5 │ ├── sea-orm-macros v0.10.5 (proc-macro) │ ├── sea-query v0.27.2 │ │ ├── sea-query-derive v0.2.0 (proc-macro) │ ├── sea-query-binder v0.2.2 │ │ ├── sea-query v0.27.2 () │ ├── sea-strum v0.23.0 │ │ └── sea-strum_macros v0.23.0 (proc-macro) │ ├── sea-orm v0.10.5 ()

Additional Information

Screenshot_20221205_100313

Screenshot_20221205_100231

Screenshot_20221205_100518

billy1624 commented 1 year ago

Hey @aadi58002, thanks for the report!! You're right, this is an error because of the recursion limit error as discussed here, https://github.com/SeaQL/seaography/pull/84#discussion_r1019035659.

I just updated the example to override the default recursion limit, https://github.com/SeaQL/seaography/pull/107.