Skip to content

Different pool, different ConnectionManager #8

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

Conversation

justmobilize
Copy link
Collaborator

@justmobilize justmobilize commented Apr 20, 2024

Update ConnectionManager, to support multiple different pools at the same time.

Testing steps:
Have a device with onboard WiFi and either a ESP32SPI or WIZNET5K
Connect both and make requests on both and see that they are from different devices.

You can also use a WIZNET5K create it's request session first, and then disconnect it and verify the other device still works.

@justmobilize justmobilize marked this pull request as ready for review April 20, 2024 21:06
@justmobilize
Copy link
Collaborator Author

@anecdata here's another one that would be awesome if you could test (with everything else in main)

@justmobilize
Copy link
Collaborator Author

Since you easily have the ability to see where requests are coming from

Copy link
Member

@anecdata anecdata left a comment

Choose a reason for hiding this comment

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

Looks good! Test code and hardware photo here.

Adafruit CircuitPython 9.1.0-beta.1 on 2024-04-19; FeatherS3 with ESP32S3
(all other libraries standard latest except for this PR #8 version of ConnectionManager)

native wifi: UnexpectedMaker FeatherS3
ethernet:    WIZnet 850io (W5500) "BFF"
esp32spi:    Adafruit Airlift Breakout

Let it loop for a while, and verified that the router view of the HTTP sessions / IPv4 addresses matched the serial print sequence.

@anecdata anecdata requested a review from a team April 25, 2024 18:13
@anecdata
Copy link
Member

Looks good from a testing perspective, maybe someone wants to code review.

@justmobilize
Copy link
Collaborator Author

@dhalbert once this is merged, I can take care of any conflicts and open up the 3x use new socketpool PRs

Copy link
Contributor

@dhalbert dhalbert left a comment

Choose a reason for hiding this comment

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

lgtm!

@dhalbert dhalbert merged commit fc33375 into adafruit:main Apr 25, 2024
1 check passed
@dhalbert
Copy link
Contributor

@justmobilize is this a bugfix version bump, or minor or major version bump, in terms of behavior?

@justmobilize
Copy link
Collaborator Author

@dhalbert I would consider it minor. It shouldn't break anything as it's only changing how pseudo private data is stored.

@justmobilize justmobilize deleted the different-pool-different-connection-manager branch April 25, 2024 21:13
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Apr 26, 2024
Updating https://github.com/adafruit/Adafruit_CircuitPython_ConnectionManager to 1.1.0 from 1.0.1:
  > Merge pull request adafruit/Adafruit_CircuitPython_ConnectionManager#8 from justmobilize/different-pool-different-connection-manager
  > Merge pull request adafruit/Adafruit_CircuitPython_ConnectionManager#9 from justmobilize/wiznet5k-ssl
  > Merge pull request adafruit/Adafruit_CircuitPython_ConnectionManager#12 from justmobilize/test-updates
  > Merge pull request adafruit/Adafruit_CircuitPython_ConnectionManager#5 from FoamyGuy/fix_circup_instruction

Updating https://github.com/adafruit/Adafruit_CircuitPython_Bundle/circuitpython_library_list.md to NA from NA:
  > Updated download stats for the libraries
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.

3 participants