-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add Reva #35
Add Reva #35
Conversation
Hm, http://revad2.docker/ocm-provider/ still prompts for BasicAuth. |
Ah, |
I could add a proxy in front that corrects this, but instead I'll try to get it to work with a branch of reva itself. |
I've added work-arounds for cs3org/reva#1962 and cs3org/reva#1752 in michielbdejong/ocm-stub@18a1f96 Now I'm having trouble seeing how I can make the contents of http://stub2.docker/ocm-provider/ change by adding |
|
OK, finally past that problem, now next one: {
shareWith: 'admin@http://revad2.docker',
name: 'Test share from stub',
providerId: '42',
owner: 'admin@https://stub1.docker',
ownerDisplayName: 'admin',
sender: 'admin@https://stub1.docker',
senderDisplayName: 'admin',
shareType: 'user',
resourceType: 'file',
protocol: { name: 'webdav', options: { sharedSecret: 'shareMe' } }
} { name: 'webdav', options: { sharedSecret: 'shareMe' } }
outgoing share created! 400 {
"code": "INVALID_PARAMETER",
"message": "missing details about resource to be shared"
} |
Reproduced on my laptop with |
See also #25 (comment) |
With michielbdejong/ocm-stub@ed0a6ef the share can now successfully be created, and the error changes to:
|
Apparently https://github.com/cs3org/reva/blob/v1.11.0/internal/http/services/ocmd/shares.go#L107-L109 is checking whether the |
Next: accept the share in reva. Should we create a reva client in node from the .proto, or use reva-cli? |
Will see how far I get with https://www.npmjs.com/package/@cs3org/node-cs3apis |
|
|
That's with 60ca2da |
Stuck on cs3org/cs3apis#140 |
OK, back in business :)
|
Next:
|
(node:84995) UnhandledPromiseRejectionWarning: TypeError: shares.getSharesList(...)[0].getReferenceId is not a function |
Added in the reva-client branch. |
Reva client
No description provided.