-
-
Notifications
You must be signed in to change notification settings - Fork 322
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
Updating Nextcloud API #803
base: master
Are you sure you want to change the base?
Conversation
Hey there if I saw it correctly the URL isn't correct. |
So are you saying it should be like this?
Also, this is where I am looking at. Basic APIs |
I checked with my local NC running Version 30.0.4 |
update url
Update url
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There the tailing / on the Url is also missing
Add /
I tested it locally and it working 👍 |
Hello.
From my very limited testing and reading this change should work but in practice it appears not to.
What I have tried.
curl -X GET https://dummy.url/ocs/v2.php/cloud/user -u "dummy:random-token-0000-bla-bla-bla" -H "OCS-APIRequest: true"
Etc.
To me that is what we should get.
I tried changing the Nextcloud.php and app.json files in Heimdall/app/SupportedApps/Nextcloud/ to include the changes I have made but it did not work. I get
General error connecting with API
I am not sure if that is because I have not done something correct locally or if this just does not work.One other thing is the Password section shows that the first 5 characters of the key is in the App Password box but this happens regardless of me putting in a key or not so is this just a visual thing and intentional?
Can someone please test and check this
Thanks.