[full-ci] Bump Reva

[kobergj]

Bumps reva

Fixes #5139
Fixes #6555

[phil-davis]

https://drone.owncloud.com/owncloud/ocis/23546/49/5

  Scenario: download a single file as different user                                                  # /drone/src/tests/acceptance/features/apiArchiver/downloadById.feature:69
    Given user "Brian" has been created with default attributes and without skeleton files            # FeatureContext::userHasBeenCreatedWithDefaultAttributesAndWithoutSkeletonFiles()
    And user "Alice" has uploaded file with content "some data" to "/textfile0.txt"                   # FeatureContext::userHasUploadedAFileWithContentTo()
    When user "Brian" downloads the archive of "/textfile0.txt" of user "Alice" using the resource id # ArchiverContext::userDownloadsTheArchiveOfItemOfUser()
    Then the HTTP status code should be "404"                                                         # FeatureContext::thenTheHTTPStatusCodeShouldBe()
      HTTP status code 200 is not the expected value 404
      Failed asserting that 200 matches expected '404'.

It seems that Brian is now able to access a file that belongs to Alice. That needs to be investigated - is it the server code, or is the test doing something unexpected?

[kobergj]

Kind of. He gets the not found error message with no 404 status code but 200 instead. I'm investigating...

[kobergj]

Needs another reva fix unforty: cs3org/reva#4010

[phil-davis]

https://github.com/kobergj/ocis/tree/BumpReva
This branch is 2 commits ahead, 13 commits behind owncloud:master.

I'm not sure if there was anything important in the most recent 13 commits, but I think that GitHub does an auto-merging anyway when it provides the branch contents to drone CI.

[kobergj]

You are right as always 😄 I rebased to current master to minimize risk.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[phil-davis]

LGTM if CI passes.