-
Notifications
You must be signed in to change notification settings - Fork 5
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
Anonymous uploads + track upload IPs #167
Conversation
Implement anonymous uploads and downloads.
Fix a bug in downloads, where an unrelated recent download would be updated instead of a new one being created.
# Conflicts: # test/database/user_test.go
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking pretty good. Only a few minor comments.
what was the bug by the way? 🙂 |
|
Add an `AnonUser` helper instead of manually creating an empty user.
Co-authored-by: Peter-Jan Brone <peter-jan@settlemint.com>
PULL REQUEST
Overview
Track anonymous uploads and downloads.
Track the IP of any uploader.
Note:
Don't worry about the anonymous download tracking - we are going to stop tracking downloads, anyway. I just wanted to keep them in sync for the sake of my own mental model. That also helped root out a bug.
Example for Visual Changes
Checklist
Review and complete the checklist to ensure that the PR is complete before assigned to an approver.
Issues Closed