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] Timeouts in Serial communication #140

Closed
cguimaraes opened this issue Jan 12, 2023 · 2 comments
Closed

[Bug] Timeouts in Serial communication #140

cguimaraes opened this issue Jan 12, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@cguimaraes
Copy link
Member

Describe the bug

Currently, timeouts are not supported in Serial transports.
Because of that, a read task might block forever while trying to read if the correspondent session was closed. Consequently, the z_close call will also block indefinitely.

To reproduce

  1. Start zenoh-pico session towards zenohd using a Serial locator.
  2. Close the session.
  3. The task closing the session will block indefinitely.

System info

N/A
@cguimaraes cguimaraes added the bug Something isn't working label Jan 12, 2023
@cguimaraes cguimaraes self-assigned this Jan 12, 2023
@cguimaraes cguimaraes removed their assignment Jan 22, 2023
@cguimaraes cguimaraes added enhancement Things could work better and removed bug Something isn't working labels Feb 1, 2023
@cguimaraes cguimaraes added bug Something isn't working and removed enhancement Things could work better labels Feb 10, 2023
@cguimaraes
Copy link
Member Author

#362 is partially addressing this issue on esp-idf port.

@cguimaraes cguimaraes mentioned this issue Mar 6, 2024
Closed
@jean-roland
Copy link
Contributor

#383 Solves this for esp-idf platform

@Mallets Mallets closed this as completed Apr 4, 2024
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

No branches or pull requests
3 participants
@cguimaraes @Mallets @jean-roland