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

Issues from PR#151 -- DTN integration #152

Open
4 tasks
spficklin opened this issue Jun 13, 2018 · 0 comments
Open
4 tasks

Issues from PR#151 -- DTN integration #152

spficklin opened this issue Jun 13, 2018 · 0 comments

Comments

@spficklin
Copy link
Collaborator

spficklin commented Jun 13, 2018

Fixes were made to allow BDSS to communicate with a DTN to transfer files. That code was merged through PR #151. However, @nawatts raised some important points that need to be addressed. See the notes on the code within the PR comments for more details. Here's a checklist of items that need addressing:

  • The temporary file code needs to be swapped out for something that doesn't create a local file.
  • The DTN function needs to support --dry-run and --transfer-report options
  • Check if URLs need quoting when sending to the subprocess function.
  • Add new DTN config options to the proper README.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant