mysql - record and check state of local files #32
No reviewers
Labels
No labels
area: devops
area: discovery
area: docs
area: livestream
area: proposal
consider soon
dependencies
Epic
good first issue
hacktoberfest
help wanted
icebox
level: 1
level: 2
level: 3
level: 4
needs: exploration
needs: grooming
needs: priority
needs: repro
needs: tech design
on hold
Osprey
priority: blocker
priority: high
priority: low
priority: medium
protocol dependent
resilience
Tom's Wishlist
type: bug
type: discussion
type: error handling
type: improvement
type: new feature
type: refactor
type: task
type: testing
unplanned
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: LBRYCommunity/spee.ch#32
Loading…
Reference in a new issue
No description provided.
Delete branch "mysql"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Added mysql table to track Files stored locally. The serve routes now check the Files table before trying to perform a 'get' on the daemon. If a local match is found based on claim_id, it checks the outpoint and either serves that local file or retrieves an updated one.
Good stuff!
What's this?
It might be a good idea to declare an extremely simple error logging class or function that simply calls
console.log
internally. That way when you want to replace it later with more sophisticated logging, you can do so easily, rather than needing to track down all the log calls.Not a blocking suggestion.
Writing all paths as relative from the root project folder rather than relative from the current file will make for later easier refactoring IMO.
This section can be simplified by avoiding repetition of
response.data.result
.(unnecessary return)
lbryHelpers.js
sounds like and sort of looks like the kind of file that is on track to accumulate a lot of functions and become a mess. Worth looking at the typical patterns for unique or business-level modal functions in this framework.✔️
✔️
may have gone a little overboard, but I did a big reorganize to address this because I agree itt was getting disorganized. This hopefully will lay the groundwork for better organization going forward. Separated out true controller functions from helper methods.
good point. will aim to address on next update.
agreed. will wait and address this with the next big update which will be focused on logging.
it's the path for the folder that the uploads will be uploaded to. I renamed it to 'PublishUploadPath'