Add mimetype to OCS Share API output

[rullzer]

Fixes #23044

Was missed during the conversion of the API and apparently no tests in place. (they are now)

@BenS89 please verify

CC: @LukasReschke @PVince81 @nickvergessen @MorrisJobke

[rullzer]

@karlitschek trivial patch but fixes some UX. Backport to 9.0.1 makes sense to me.

[karlitschek]

backport makes sense 👍

[BenS89]

@rullzer that did the trick. Thanks for fixing the issue with this PR.

[rullzer]

stable9 in #23060

[MorrisJobke]

👍

[nickvergessen]

Can we add that to the integration test?

[DeepDiver1975]

Can we add that to the integration test?

@rullzer

[rullzer]

Yes good point. let me add that.

[rullzer]

Intergration tests added.

[MorrisJobke]

Feature: federated

  Background:                   # /ssd/jenkins/workspace/ocs-api-integration-tests-ci/build/integration/federation_features/federated.feature:2
    Given using api version "1" # FederationContext::usingApiVersion()

  Scenario: Federate share a file with another server                                                    # /ssd/jenkins/workspace/ocs-api-integration-tests-ci/build/integration/federation_features/federated.feature:5
    Given Using server "REMOTE"                                                                          # FederationContext::usingServer()
[Thu Mar 10 08:34:45 2016] ::1:38866 [404]: /ocs/v2.php/cloud/users/user1
[Thu Mar 10 08:34:46 2016] ::1:38880 [200]: /ocs/v1.php/cloud/users
[Thu Mar 10 08:34:48 2016] ::1:38888 [200]: /ocs/v1.php/cloud/users/user1
[Thu Mar 10 08:34:49 2016] ::1:38900 [200]: /ocs/v2.php/cloud/users/user1
    And user "user1" exists                                                                              # FederationContext::assureUserExists()
    And Using server "LOCAL"                                                                             # FederationContext::usingServer()
[Thu Mar 10 08:34:49 2016] ::1:33709 [404]: /ocs/v2.php/cloud/users/user0
[Thu Mar 10 08:34:50 2016] ::1:33715 [200]: /ocs/v1.php/cloud/users
Build timed out (after 20 minutes). Marking the build as aborted.
Build was aborted
Regular expression run condition: Expression=[origin/master|origin/stable\d+(\.\d+)?], Label=[origin/fix_23044]
Run condition [Regular expression match] preventing perform for step [[E-mail Notification]]
Recording test results
    And user "user0" exists                                                                              # FederationContext::assureUserExists()
      cURL error 52: Empty reply from server (GuzzleHttp\Exception\ConnectException)
    When User "user0" from server "LOCAL" shares "/textfile0.txt" with user "user1" from server "REMOTE" # FederationContext::federateSharing()
    Then the OCS status code should be "100"                                                             # FederationContext::theOCSStatusCodeShouldBe()
    And the HTTP status code should be "200"                                                             # FederationContext::theHTTPStatusCodeShouldBe()
  │
  ╳  cURL error 56: Recv failure: Connection reset by peer (GuzzleHttp\Exception\RequestException)
  │
  └─ @AfterScenario # FederationContext::cleanupUsers()

@rullzer Simply rerun? Or is this an actual failure?

[rullzer]

Build timed out (after 20 minutes). Marking the build as aborted.

@MorrisJobke it is a timeout... @DeepDiver1975 is already notified.

[DeepDiver1975]

@MorrisJobke it is a timeout... @DeepDiver1975 is already notified.

incremented to 30 minutes

[rullzer]

Thanks @DeepDiver1975

Rebased to retrigger CI

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.