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

Simple http-auth added #2

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

Simple http-auth added #2

wants to merge 8 commits into from

Conversation

jwd83
Copy link

@jwd83 jwd83 commented Oct 15, 2016

Hey there, I've recently been learning Go and simultaneously found a use for this code to quickly share some large files with a friend of mine. I wanted to try to add a basic http-auth function to provide a minimum level of security for times I'd like to make this public facing breifly. I imagine a .gitignore could/should be added. In any event would love to hear your thoughts or if I should simply keep this as my own fork.

@jwd83
Copy link
Author

jwd83 commented Sep 29, 2017

Hey there, do you think you will ever come back to this project? I still find it handy from time to time but haven't spent any more time learning go. How difficult would it be to tack on https support?

@kidoman
Copy link
Owner

kidoman commented Sep 29, 2017

Hey, wow I had completely missed this. Let me see if we can do something about https. The obvious approach would be something like LE (via caddy) but seems like a lot of work for a temporary service like this. Did you have anything else in mind.

@jwd83
Copy link
Author

jwd83 commented Dec 18, 2017

Hey there, I also haven't checked back in on this recently. When I tried to setup this project again a few weeks ago to move some backups I ran into what I think was a dependency problem setting it up and went and found another solution in the mean time. I think this is still a great little project to get files from A to B and adding some level of https (even non lets encrypt just self signed?) would he handy.

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