Skip to content

UDP server requires opening socket on chip during bind #39

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

Merged
merged 3 commits into from
Jul 22, 2021

Conversation

obaltzer
Copy link
Contributor

UDP server support requires the socket on the chip to be opened during
bind. For TCP the socket_listen function handles this and now bind calls
an equivalent socket_listen_udp if the socket type is SOCK_DGRAM.

UDP server support requires the socket on the chip to be opened during
bind. For TCP the socket_listen function handles this and now bind calls
an equivalent socket_listen_udp if the socket type is SOCK_DGRAM.
@ladyada ladyada requested a review from a team July 21, 2021 14:21
@obaltzer obaltzer requested a review from brentru July 21, 2021 15:49
@obaltzer obaltzer requested a review from brentru July 21, 2021 17:32
Copy link
Member

@brentru brentru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - thank you for the fix!

@brentru brentru merged commit 2ba7082 into adafruit:main Jul 22, 2021
@obaltzer obaltzer deleted the open-udp-server-socket branch July 22, 2021 14:24
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jul 28, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_DisplayIO_SH1106 to 1.1.0 from 1.0.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_DisplayIO_SH1106#4 from lesamouraipourpre/max-size

Updating https://github.com/adafruit/Adafruit_CircuitPython_ESP_ATcontrol to 0.5.6 from 0.5.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#40 from CytronTechnologies/increase-socket-connect-timeout
  > Merge pull request adafruit/Adafruit_CircuitPython_ESP_ATcontrol#39 from CytronTechnologies/fix-compatibility-issue
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template
  > "Increase duplicate code check threshold "

Updating https://github.com/adafruit/Adafruit_CircuitPython_MLX90640 to 1.2.4 from 1.2.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_MLX90640#25 from lesamouraipourpre/remove-max-glyphs
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template
  > "Increase duplicate code check threshold "

Updating https://github.com/adafruit/Adafruit_CircuitPython_Wiznet5k to 1.11.0 from 1.10.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_Wiznet5k#40 from AdamCummick/snrx_rd-rollover-bug
  > Merge pull request adafruit/Adafruit_CircuitPython_Wiznet5k#39 from obaltzer/open-udp-server-socket

Updating https://github.com/adafruit/Adafruit_CircuitPython_CursorControl to 2.4.0 from 2.3.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_CursorControl#25 from lesamouraipourpre/remove-max-glyphs

Updating https://github.com/adafruit/Adafruit_CircuitPython_Debug_I2C to 1.2.4 from 1.2.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_Debug_I2C#9 from adafruit/fix_nonerror
  > Moved CI to Python 3.7
  > Changed readme link
  > Added help text and problem matcher
  > Added pull request template
  > "Increase duplicate code check threshold "

Updating https://github.com/adafruit/Adafruit_CircuitPython_Display_Shapes to 2.3.0 from 2.2.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_Display_Shapes#39 from lesamouraipourpre/remove-max-glyphs

Updating https://github.com/adafruit/Adafruit_CircuitPython_HID to 5.1.0 from 5.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_HID#74 from andreblue/main
  > Moved default branch to main

Updating https://github.com/adafruit/Adafruit_CircuitPython_PyBadger to 3.5.0 from 3.4.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_PyBadger#47 from lesamouraipourpre/remove-max-glyphs

Updating https://github.com/adafruit/Adafruit_CircuitPython_Simple_Text_Display to 1.1.0 from 1.0.2:
  > Merge pull request adafruit/Adafruit_CircuitPython_Simple_Text_Display#4 from lesamouraipourpre/remove-max-glyphs
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 this pull request may close these issues.

2 participants