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

[Bug]: In NetGear client.recv() stucking in a some non returnable loop #374

Closed
3 tasks done
sourabmaity opened this issue Aug 31, 2023 · 4 comments · Fixed by #378
Closed
3 tasks done

[Bug]: In NetGear client.recv() stucking in a some non returnable loop #374

sourabmaity opened this issue Aug 31, 2023 · 4 comments · Fixed by #378
Assignees
Labels
BUG 🐛 Vidgear api's error, flaw or fault SOLVED 🏁 This issue/PR is resolved now. Goal Achieved!

Comments

@sourabmaity
Copy link

Description

I create a simple python code for client and server

client side in a continuous while loop where its receiving image using client.recv()

server side also in a continuous loop where its sends images to netgear specific address and port

But I face a problem where client side client.recv() function stuck in some loop I think its not returning any thing when server face some unsurten network issue and disconnected .

How to solve this ?

Issue Checklist

  • I have searched open or closed issues for my problem and found nothing related or helpful.
  • I have read the Documentation and found nothing related to my problem.
  • I've read the Issue Guidelines and wholeheartedly agree.

Expected behaviour

its has a request_timeout of 5 sec , so its must return None after 5 sec.

Actual behaviour

Not executing next line of code after client.recv() seems its stuck in a continuous loop

Steps to reproduce

1> I found this bug by disconnecting the server in the middle of run.
2> or disconnecting internet in the middle of run

Terminal log output

No response

Python Code(Optional)

No response

VidGear Version

0.3.1

Python version

3.11.4

OpenCV version

4.8.0

Operating System version

windows10

Any other Relevant Information?

No response

@sourabmaity sourabmaity added the BUG 🐛 Vidgear api's error, flaw or fault label Aug 31, 2023
@welcome
Copy link

welcome bot commented Aug 31, 2023

Thanks for opening this issue, a maintainer will get back to you shortly!

In the meantime:

  • Read our Issue Guidelines, and update your issue accordingly. Please note that your issue will be fixed much faster if you spend about half an hour preparing it, including the exact reproduction steps and a demo.
  • Go comprehensively through our dedicated FAQ & Troubleshooting section.
  • For any quick questions and typos, please refrain from opening an issue, as you can reach us on Gitter community channel.

@sourabmaity sourabmaity changed the title [Bug]: In NetGear client.recv() stucking in a some non return able loop [Bug]: In NetGear client.recv() stucking in a some non returnable loop Aug 31, 2023
@abhiTronix
Copy link
Owner

@sourabmaity Which pattern are you using?

@abhiTronix abhiTronix added MISSING : INFORMATION 🔍 Requested proper information/evidence about the Issue/PR in context. WAITING FOR RESPONSE ⏳ Waiting for the user response. BUG UNCONFIRMED 🙅 Bug is still uncomfirmed! labels Sep 2, 2023
@abhiTronix
Copy link
Owner

Possible duplicate of #361

@abhiTronix abhiTronix added DUPLICATE ♻️ Similar issue or PR already exists and removed BUG UNCONFIRMED 🙅 Bug is still uncomfirmed! labels Sep 2, 2023
abhiTronix added a commit that referenced this issue Sep 2, 2023
…Fixes #374)

- ✨ Added new `subscriber_timeout` integer optional parameter to support timeout with `pattern=2` or PUBLISHER-SUBSCRIBER pattern.
  - 🧑‍💻 Receiver will exit safely if timeout defined(any value(in milliseconds) > 0), and timeout occurs in Receiver Mode with `pattern=2`.
  - 🎨 Note: Default behavior still is to block the thread till infinite time.
- 🔊 Updated logging.
@abhiTronix abhiTronix added WORK IN PROGRESS 🚧 currently been worked on. and removed DUPLICATE ♻️ Similar issue or PR already exists MISSING : INFORMATION 🔍 Requested proper information/evidence about the Issue/PR in context. WAITING FOR RESPONSE ⏳ Waiting for the user response. labels Sep 2, 2023
@abhiTronix abhiTronix linked a pull request Sep 4, 2023 that will close this issue
6 tasks
@abhiTronix
Copy link
Owner

Successfully resolved and merged in commit 3c0dd5d

@abhiTronix abhiTronix added SOLVED 🏁 This issue/PR is resolved now. Goal Achieved! and removed WORK IN PROGRESS 🚧 currently been worked on. labels Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BUG 🐛 Vidgear api's error, flaw or fault SOLVED 🏁 This issue/PR is resolved now. Goal Achieved!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants