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

Bug: unexpected EOF on TestSurrealDBSuite/gorilla/TestUpdate #124

Closed
2 tasks done
ElecTwix opened this issue Jan 28, 2024 · 1 comment · Fixed by #125
Closed
2 tasks done

Bug: unexpected EOF on TestSurrealDBSuite/gorilla/TestUpdate #124

ElecTwix opened this issue Jan 28, 2024 · 1 comment · Fixed by #125
Labels
bug Something isn't working

Comments

@ElecTwix
Copy link
Contributor

Describe the bug

While working on #119 noticed while TestSurrealDBSuite/gorilla/TestUpdate running getting unexpected EOF

Steps to reproduce

  1. Print or Log the error on pkg/conn/gorilla/gorilla.go:226
  2. Run Tests
  3. Get Error

Expected behaviour

Expected to not get this error

SurrealDB version

1.1.1 for linux on x86_64

Contact Details

root@electwix.dev

Is there an existing issue for this?

  • I have searched the existing issues

Code of Conduct

  • I agree to follow this project's Code of Conduct
@ElecTwix ElecTwix added the bug Something isn't working label Jan 28, 2024
ElecTwix added a commit to ElecTwix/surrealdb.go that referenced this issue Jan 30, 2024
@ElecTwix
Fix surrealdb#124
4d2fa21 
Added log printer after tearing down tests for showing network errors
@ElecTwix ElecTwix mentioned this issue Jan 30, 2024
Merged
ElecTwix added a commit to ElecTwix/surrealdb.go that referenced this issue Jan 30, 2024
@ElecTwix
Fix surrealdb#124
fb3ff7e 
Added log printer after tearing down tests for showing network errors
@ElecTwix
Copy link
Contributor Author

ElecTwix commented Jan 30, 2024 •
edited
Loading

to be noted: this happened because graceful shutdown does not exist in Gorilla WS

@timpratim timpratim assigned timpratim and unassigned timpratim Feb 5, 2024
timpratim pushed a commit that referenced this issue Feb 5, 2024
@ElecTwix
Fix #124 (#125)
7c2584a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix #124 ElecTwix/surrealdb.go
2 participants
@timpratim @ElecTwix