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

Add super simple http file server #47

Merged
merged 3 commits into from
Jul 11, 2024
Merged

Add super simple http file server #47

merged 3 commits into from
Jul 11, 2024

Conversation

elv-nate
Copy link
Contributor

Might be a little bit too simple, but it felt like the right spot to put something like this. Basically a very thin wrapper around go's http.FileServer. Probably not safe for production use, but good for testing.

@elv-nate elv-nate requested a review from elv-gilles July 10, 2024 22:28
@elv-nate elv-nate self-assigned this Jul 10, 2024
util/netutil/file_server.go Outdated Show resolved Hide resolved
util/netutil/file_server.go Outdated Show resolved Hide resolved
@elv-nate
Copy link
Contributor Author

@elv-gilles Super helpful suggestions, thanks. Makes it a lot more robust.

@elv-nate elv-nate merged commit bbd3ce7 into master Jul 11, 2024
1 check passed
@elv-nate elv-nate deleted the add-file-http-server branch July 11, 2024 18:59
@elv-nate elv-nate restored the add-file-http-server branch July 11, 2024 18:59
@elv-nate
Copy link
Contributor Author

elv-nate commented Jul 11, 2024

Restored branch to not immediately break branch-builds on https://github.com/qluvio/content-fabric/pull/2819. I'll delete the branch after updating.

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.

2 participants