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

Support external fetch transports via plug-in architecture. #37

Closed
mikedarcy opened this issue Oct 18, 2019 · 1 comment
Closed

Support external fetch transports via plug-in architecture. #37

mikedarcy opened this issue Oct 18, 2019 · 1 comment
Assignees
@mikedarcy
Labels
enhancement

Comments

@mikedarcy
Copy link
Collaborator

The current set of fetch transports is basically hard-wired into the code, whereby the only way to add to (or override) the default set of handlers is to modify the bdbag.fetch.transports module source code directly. This has proven to be too limiting in practice.

Refactoring the bdbag.fetch.transports module so that it supports run-time loading of externally provided fetch transport providers (similar to how the bdbag.fetch.resolvers module works) should be sufficient address this.
@mikedarcy mikedarcy self-assigned this Oct 18, 2019
mikedarcy added a commit that referenced this issue Oct 19, 2019
@mikedarcy
Overhaul fetch transport module to support plug-in transport handlers…
88ed9e3 
…. Addresses #37.

Improve unit test coverage.
@mikedarcy mikedarcy mentioned this issue Oct 24, 2019
Merged
@mikedarcy
Copy link
Collaborator Author

Support for this has been implemented, merged to master, and will be released in the upcoming 1.6.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikedarcy mikedarcy

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mikedarcy