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.

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

Postgres example Broken with reached the recursion limit error #106

Closed
aadi58002 opened this issue Dec 5, 2022 · 1 comment · Fixed by #107
Closed

Postgres example Broken with reached the recursion limit error #106

aadi58002 opened this issue Dec 5, 2022 · 1 comment · Fixed by #107
Assignees

Comments

@aadi58002
Copy link

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:[email protected]/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 billy1624 mentioned this issue Dec 6, 2022
1 task
@billy1624
Copy link
Member

Hey @aadi58002, thanks for the report!! You're right, this is an error because of the recursion limit error as discussed here, #84 (comment).

I just updated the example to override the default recursion limit, #107.

@billy1624 billy1624 self-assigned this Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants