Showing 363 of 363 total issues
TODO found Open
Open
// TODO: Add in sync operation to update object record in COMS DB?
- Exclude checks
Expected a newline at the end of the file. Open
Open
<!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
- Exclude checks
TODO found Open
Open
// TODO: use req.currentObject.bucketId
- Exclude checks
Unexpected additional newlines at the end of the file. Open
Open
<!-- If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... -->
- Exclude checks
TODO found Open
Open
// TODO: Proper 304 caching logic (with If-Modified-Since header support)
- Exclude checks
TODO found Open
Open
// TODO: Consider adding API param support for Server Side Encryption
- Exclude checks
TODO found Open
Open
// TODO: consider updating metadata here instead of the controller
- Exclude checks
TODO found Open
Open
// TODO: sync with current versions in S3
- Exclude checks
TODO found Open
Open
// TODO: Consider support for filtering by set of permissions?
- Exclude checks
TODO found Open
Open
// TODO: Simplify by changing from arrow function to property
- Exclude checks
TODO found Open
Open
// TODO: Add validation to reject unexpected query parameters
- Exclude checks
TODO found Open
Open
// TODO: Proper 304 caching logic (with If-Modified-Since header support)
- Exclude checks
TODO found Open
Open
// TODO: Consider adding an "all" mode for checking through all known objects and buckets for job enumeration
- Exclude checks
TODO found Open
Open
// TODO: consider doing one bulk delete query instead of using forEach
- Exclude checks
TODO found Open
Open
# TODO: does pr-123 user need to own database pr-123 in order to connect run knex migrations?
- Exclude checks
TODO found Open
Open
// TODO: Figure out what's killing and returning a 400 in response stack
- Exclude checks
TODO found Open
Open
// TODO: Consider adding 'x-coms-public' and 'x-coms-path' headers into API spec?
- Exclude checks
TODO found Open
Open
// TODO: use isLatest where possible
- Exclude checks
TODO found Open
Open
// TODO: Consider conditionally skipping when identityId is undefined?
- Exclude checks
TODO found Open
Open
// TODO: Consider adding API param support for Server Side Encryption
- Exclude checks