bcgov / common-object-management-service

A microservice for managing access control to S3 Objects
https://bcgov.github.io/common-object-management-service/
Apache License 2.0
6 stars 9 forks source link

Allow Content-Disposition types other than attachment #271

Closed norrisng-bc closed 2 months ago

norrisng-bc commented 2 months ago

Description

Content-Disposition types other than attachment are now allowed.

The following are still required, however:

RFC 6266 had no such requirement where the type must be attachment.

https://apps.nrs.gov.bc.ca/int/jira/browse/SHOWCASE-3716

Types of changes

Breaking change (fix or feature that would cause existing functionality to change)

Checklist

Further comments

COMS didn't even really use the disposition type in the first place, but it still needs the Content-Disposition header because that's where it gets the filename.

github-actions[bot] commented 2 months ago

Coverage Report

Totals Coverage
Statements: 61.06% ( 3045 / 4987 )
Methods: 50.99% ( 335 / 657 )
Lines: 68.14% ( 1831 / 2687 )
Branches: 53.5% ( 879 / 1643 )
codeclimate[bot] commented 2 months ago

Code Climate has analyzed commit 82dfdc14 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 68.1% (0.0% change).

View more on Code Climate.