Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[OCIS] invalid status code on OCS requests for object that doesn't exist #250

Closed
dpakach opened this issue Oct 7, 2020 · 4 comments
Closed

Comments

@dpakach
Copy link

dpakach commented Oct 7, 2020

the ocs status code when requesing user that doesn't exists should be 404 but returns 998

❯ curl -X GET https://localhost:9200/ocs/v2.php/cloud/users/michaelsdsfs -ks | xmllint --format -
<?xml version="1.0" encoding="UTF-8"?>
<ocs>
  <meta>
    <status>error</status>
    <statuscode>998</statuscode>
    <message>The requested user could not be found</message>
  </meta>
</ocs>

on OC10

❯ curl -XGET http://localhost/fed/ocs/v2.php/cloud/users/michaelsdsfs -u admin:admin -ks
<?xml version="1.0"?>
<ocs>
 <meta>
  <status>failure</status>
  <statuscode>404</statuscode>
  <message>The requested user could not be found</message>
 </meta>
 <data/>
</ocs>
@phil-davis
Copy link

@dpakach your OCIS example is with v1.php and the oC10 example is with v2.php - those do have different ocs status codes anyway.

@dpakach
Copy link
Author

dpakach commented Oct 7, 2020

@dpakach your OCIS example is with v1.php and the oC10 example is with v2.php - those do have different ocs status codes anyway.

Yes, updated now. But the result is still same

@dpakach dpakach changed the title [OCIS] invalid status code when requesting user info for user that doesn't exist [OCIS] invalid status code on OCS requests for object that doesn't exist Jun 2, 2021
@dpakach
Copy link
Author

dpakach commented Jun 2, 2021

Also same result for shares

❯ curl https://localhost:9200/ocs/v2.php/apps/files_sharing/api/v1/shares/716sdfdsf -u admin:admin -k
<?xml version="1.0" encoding="UTF-8"?>
<ocs><meta><status>error</status><statuscode>998</statuscode><message>share not found</message></meta></ocs>%  

@saw-jan
Copy link
Member

saw-jan commented Feb 21, 2024

Fixed by cs3org/reva#4529 and owncloud/ocis#8412

@saw-jan saw-jan closed this as completed Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants