-
Notifications
You must be signed in to change notification settings - Fork 0
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
listAllDrivesWithHttpInfo function fails to catch and null response exception #4
listAllDrivesWithHttpInfo function fails to catch and null response exception #4
Comments
The problem here is if the server returns a |
should be fixed by OpenAPITools/openapi-generator#16879 |
@S-Panta please retest, and check if the issue is fixed |
Somehow, the body of the |
@S-Panta exactly and in the current version of this library (after OpenAPITools/openapi-generator#16879) the function should not return |
I used the latest generation and the response is still |
The current image doesn't include fixed PR. I made bump PR : owncloud/libre-graph-api#143 |
The bump did not do anything, it did not produce any different output |
the issue was that we haven't applied the fix of OpenAPITools/openapi-generator#17120 in the template used in the generator, should be done by owncloud/libre-graph-api#146 |
@S-Panta please confirm if the issue is really fixed |
It is fixed. |
If the URL is in the wrong format,
response
would benull
and this function doesn't throw any exception but returnsnull
.response
asnull
makes no sense. Thus the function should handle this exception.Note: I have used
[ocis-php-sdk]
to reproduce the error.https://github.com/owncloud/ocis-php-sdk
Steps to reproduce
Create an Ocis object using the service Url with an extra slash in it and an access token.
The response will be returned as
null
.Expected result
The function should handle the error.
The text was updated successfully, but these errors were encountered: