-
Notifications
You must be signed in to change notification settings - Fork 115
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
added support for public links #241
Conversation
Great addition, thanks ! Would be good to add some unit tests. A slight worry might be that not all the session operations will work in anonymous mode like creating shares when connecting through that session. Might not be a big issue as calling those would result in an HTTPResponseError with code 403 or 401. |
it would also be nice to add an example usage in the README :-) |
that's more tests than I expected, awesome! |
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.
👍
Thanks! I forgot to push the updated examples:
|
@mrwunderbar666 mind sending in another PR ? else I can copy paste this later on |
Adds support to manipulate files in a publicly shared folder (with and without password). As described in #12
anon_session = owncloud.Client.from_public_link(public_link, password)