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

eos: support arbitrary transport class #1643

Merged
merged 3 commits into from
Jun 3, 2022
Merged

Conversation

bewing
Copy link
Member

@bewing bewing commented May 20, 2022

Allow passing of a subclass of EapiConnection as the transport, either
as a class object or a string that can be imported as a class.

@mirceaulinic mirceaulinic added this to the 4.0.0 milestone May 21, 2022
@bewing bewing force-pushed the eos_arbitrary_transport branch from 71c3b51 to 7222c18 Compare May 23, 2022 18:27
Allow passing of a subclass of EapiConnection as the transport, either
as a class object or a string that can be imported as a class.
@bewing bewing force-pushed the eos_arbitrary_transport branch from 7222c18 to defc367 Compare May 23, 2022 18:38
bewing added a commit that referenced this pull request May 24, 2022
In conjunction with #1643, if an arbitrary transport class is stateful,
we should be trying to maintain state (exiting config mode after
entering, closing when closing, etc).
@bewing bewing mentioned this pull request May 24, 2022
bewing added a commit that referenced this pull request May 24, 2022
In conjunction with #1643, if an arbitrary transport class is stateful,
we should be trying to maintain state (exiting config mode after
entering, closing when closing, etc).
bewing added a commit that referenced this pull request May 24, 2022
In conjunction with #1643, if an arbitrary transport class is stateful,
we should be trying to maintain state (exiting config mode after
entering, closing when closing, etc).
@mirceaulinic mirceaulinic merged commit dde6a59 into develop Jun 3, 2022
@mirceaulinic mirceaulinic deleted the eos_arbitrary_transport branch June 3, 2022 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants