bcgov/common-object-management-service

View on GitHub

Showing 363 of 363 total issues

TODO found
Open

          // TODO: Add in sync operation to update object record in COMS DB?
Severity: Minor
Found in app/src/controllers/object.js by fixme

Expected a newline at the end of the file.
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... -->

TODO found
Open

        // TODO: use req.currentObject.bucketId
Severity: Minor
Found in app/src/controllers/object.js by fixme

Unexpected additional newlines at the end of the file.
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... -->

TODO found
Open

        // TODO: Proper 304 caching logic (with If-Modified-Since header support)
Severity: Minor
Found in app/src/controllers/object.js by fixme

TODO found
Open

        // TODO: Consider adding API param support for Server Side Encryption
Severity: Minor
Found in app/src/services/storage.js by fixme

TODO found
Open

      // TODO: consider updating metadata here instead of the controller
Severity: Minor
Found in app/src/services/version.js by fixme

TODO found
Open

      // TODO: sync with current versions in S3
Severity: Minor
Found in app/src/controllers/object.js by fixme

TODO found
Open

    // TODO: Consider support for filtering by set of permissions?
Severity: Minor
Found in app/src/controllers/object.js by fixme

TODO found
Open

  // TODO: Simplify by changing from arrow function to property
Severity: Minor
Found in app/src/validators/common.js by fixme

TODO found
Open

  // TODO: Add validation to reject unexpected query parameters
Severity: Minor
Found in app/src/routes/v1/object.js by fixme

TODO found
Open

      // TODO: Proper 304 caching logic (with If-Modified-Since header support)
Severity: Minor
Found in app/src/controllers/object.js by fixme

TODO found
Open

      // TODO: Consider adding an "all" mode for checking through all known objects and buckets for job enumeration
Severity: Minor
Found in app/src/controllers/sync.js by fixme

TODO found
Open

      // TODO: consider doing one bulk delete query instead of using forEach
Severity: Minor
Found in app/src/services/metadata.js by fixme

TODO found
Open

      # TODO: does pr-123 user need to own database pr-123 in order to connect run knex migrations?
Severity: Minor
Found in .github/workflows/on-pr-opened.yaml by fixme

TODO found
Open

    // TODO: Figure out what's killing and returning a 400 in response stack
Severity: Minor
Found in app/src/middleware/upload.js by fixme

TODO found
Open

    // TODO: Consider adding 'x-coms-public' and 'x-coms-path' headers into API spec?
Severity: Minor
Found in app/src/controllers/object.js by fixme

TODO found
Open

          // TODO: use isLatest where possible
Severity: Minor
Found in app/src/services/version.js by fixme

TODO found
Open

    // TODO: Consider conditionally skipping when identityId is undefined?
Severity: Minor
Found in app/src/services/user.js by fixme

TODO found
Open

      // TODO: Consider adding API param support for Server Side Encryption
Severity: Minor
Found in app/src/services/storage.js by fixme
Severity
Category
Status
Source
Language