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

Debug/marshall/file upload corruption #3

Open
wants to merge 58 commits into
base: master
Choose a base branch
from

Conversation

fedeberco
Copy link

No description provided.

fclairamb and others added 30 commits September 1, 2017 13:24
first draft
second try
Allow passing in an existing net.Listener and a public ip resolver.
This version should be fully able to run on any architecture.

- Removing all the files related to deployment to make everything fit in one binary and allow test on any archi easier
- Allowing to only load the binary to generate a conf file (without starting the server)
- Using the binary to prepare the docker conf file
- Sample drivers now creates a self-signed TLS certificate instead of loading one
- Changed the arg parsing logic (change of heart)
…pserver into mlst-support

# Conflicts:
#	server/driver.go
…pserver into mlst-support

# Conflicts:
#	server/driver.go
Allowing to test it on Windows / MacOS / Linux
fclairamb and others added 28 commits January 16, 2018 01:39
Changes are:
- Checking the file closure
- Checking the file seek
- Sharing the code between the download and upload code
- add MLSx validation test
- Disabling bogus empty lines

Thanks to @backlin for reporting this issue and @mgenov for the code review.
Modify 226 transfer message to match what a vendor expects.
Don't think this should neccesarily end up in mainline.
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