feat: Add route that sets projectRestrictedViewSetting size (DEV-2304)

[mpro7]

Pull Request Checklist

Task Description/Number

Issue Number: DEV-2304

Basic Requirements

Please check if your PR fulfills the following requirements:

• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• fix: represents bug fixes
• refactor: represents production code refactoring
• feat: represents a new feature
• docs: documentation changes (no production code change)
• chore: maintenance tasks (no production code change)
• test: all about tests: adding, refactoring tests (no production code change)
• other... Please describe:

Does this PR introduce a breaking change?

• Yes
• No
• Maybe (not 100% sure => check with FE)

Does this PR change client-test-data?

• Yes (don't forget to update the JS-LIB team about the change)
• No

[linear]

DEV-2304 Change default value of the param projectRestrictedViewSize

Reported by Loïc (DCSR, Lausanne):

Hello everybody,

It has been mentioned many times but I can’t a trace of it, so here it comes again:

making default RestrictedViewSettings being "!512,512" barely makes sense.

Only when you look at it in its whole, it is blurred because the full resolution of, let’s say 5120x5120 pixels is reduced to 521x512, then the resolution is reduced to 1%.

When you zoom in half, looking at a picture of 2560x2560 then the resolution is reduced to 4%, zoom a bit more to 1280x1280 the resolution is reduced to 16% of the images, some faces are recognizable, text readable.

And of course, when you are able to zoom in to get a portion of 512x512 pixel, then you see that portion at full resolution!

Just walk your looking glass around and see it all!

So, the default should not be “!512x512” but “pct:1”, the resolution of the image displayed is 1% of the original resolution, whatever the size of the part the image.

Best regards,
Loïc.

https://dasch.atlassian.net/browse/DSQ-211

Related issue: DEV-991

13e21433-568a-4817-8a39-b0effe454ec7

[codecov]

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (12402f3) 18.00% compared to head (2832866) 22.23%.
Report is 74 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2794      +/-   ##
==========================================
+ Coverage   18.00%   22.23%   +4.23%     
==========================================
  Files         281      244      -37     
  Lines       28899    23123    -5776     
==========================================
- Hits         5202     5141      -61     
+ Misses      23697    17982    -5715     

see 171 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[linear]

DEV-2304 Change default value of the param projectRestrictedViewSize

the current default value "!512x512" should be changed to "pct:1"

see DEV-2269 for more detail

[seakayone]

User input must always be sanitized and validated. You cannot just simply take the users String value and put it into a query template. This opens the door for all kind of bad things, e.g. attacks and invalid data being persisted.

Make sure the user input is:

1. sanitized : the new endpoint is not open for a malicious SPARQL injection
2. validated: the size value is valid. AFAIU this must be a IIIF size expression

Add documentation for the size parameter. Also check your current documentation its example is GET whilst the endpoint is POST.