41 Commits (a2572ffec79f7bd2fa7aa29ed48d9b011836759d)

Author	SHA1	Message	Date
Lukas Reschke	933f60e314	Update author information ... Probably nice for the people that contributed to 9.0 to see themselves in the AUTHORS file :)	10 years ago
Lukas Reschke	8f95d69954	Fix tests	10 years ago
Robin Appelman	acd8c72d3d	add tests	10 years ago
Roeland Jago Douma	359c62d90e	Fix unit tests	10 years ago
Roeland Jago Douma	623a0e4637	Example hook check	10 years ago
Roeland Jago Douma	38cbfed476	Fix unit test	10 years ago
Roeland Jago Douma	453ea6698d	[Share 2.0] Fix public link sharing	10 years ago
Roeland Jago Douma	4d7130ad31	[Share 2.0] Add exceptions to OCP	10 years ago
Roeland Jago Douma	34e912ab6b	[Share 2.0] Fix interfaces and comments ... * Made comments more clear * Removed unneeded methods * IShares shareTime is now a proper DateTime object * IShares getPath -> getNode & setPath -> setNode * Fix unit tests	10 years ago
Roeland Jago Douma	185b9c6edd	[Share 2.0] Move IShare to OCP	10 years ago
Roeland Jago Douma	18421e7e68	Directly get from the server container ... * Updated unit tests	10 years ago
Roeland Jago Douma	8734ebe505	[Share 2.0] Make link share download use share manager	10 years ago
Roeland Jago Douma	717697313b	[Share 2.0] Move tests over the sharemanager ... Nice side effect... pure unit tests!	10 years ago
Thomas Müller	682821c71e	Happy new year!	10 years ago
Roeland Jago Douma	98c4951f45	getLowStrengthGenerator does not do anything anymore	10 years ago
Roeland Jago Douma	1f715289bf	Removed deprecated function OC_User::deleteUser ... Replaced with proper OCP calls	10 years ago
Roeland Jago Douma	835911bce5	Removed deprecated private OC_User::createUser ... All function calls are replaced with the recommended (which was already the body of the function).	10 years ago
Björn Schießle	c4b65170f2	show display name but internally use the user name	10 years ago
Thomas Müller	757aa45db7	Adding more DB groups to test suits ... Adding more DB groups to test suits	10 years ago
Lukas Reschke	8f09d5b67c	Update license headers	10 years ago
Lukas Reschke	13e817e901	Throw exception on `getPath` if file does not exist ... Currently the `getPath` methods returned `NULL` in case when a file with the specified ID does not exist. This however mandates that developers are checking for the `NULL` case and if they do not the door for bugs with all kind of impact is widely opened. This is especially harmful if used in context with Views where the final result is limited based on the result of `getPath`, if `getPath` returns `NULL` PHP type juggles this to an empty string resulting in all possible kind of bugs. While one could argue that this is a misusage of the API the fact is that it is very often misused and an exception will trigger an immediate stop of execution as well as log this behaviour and show a pretty error page. I also adjusted some usages where I believe that we need to catch these errors, in most cases this is though simply an error that should hard-fail.	10 years ago
Morris Jobke	b945d71384	update licence headers via script	10 years ago
Roeland Jago Douma	3d2acb5003	sharingcheckmiddleware now handles externalshares as well ... Added new annotations for the externalsharescontroller class * @NoOutgoingFederatedSharingRequired * @NoIncomingFederatedSharingRequired By default both are required for all functions in the externalSharesController. A proper exception is thrown and then a 405 is returned instead of the default error page. Since it is only an API endpoint this makes more sense. Unit tests added and updated	10 years ago
Lukas Reschke	0a1d551090	Use IClientService to check for remote ownCloud instances ... 1. Allows to set a timeout (though still not perfect but way better than before) 2. Allows to have unit tests 3. I also added unit tests for the existing controller code 4. Corrected PHPDoc on IClient	11 years ago
Morris Jobke	f63915d0c8	update license headers and authors	11 years ago
Morris Jobke	7550bf0917	Disable video preview if previews are disabled	11 years ago
Scrutinizer Auto-Fixer	fdbc21fc6c	Scrutinizer Auto-Fixes ... This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com	11 years ago
Robin Appelman	415f16eb24	fix tests	11 years ago
Robin Appelman	dd9601ae8f	fix merge	11 years ago
Jenkins for ownCloud	b585d87d9d	Update license headers	11 years ago
Lukas Reschke	e2453d78c0	Properly catch whether a share is `null` ... Despite it's PHPDoc the function might return `null` which was not properly catched and thus in some situations the share was resolved to the sharing users root directory. To test this perform the following steps: * Share file in owncloud 7 (7.0.4.2) * Delete the parent folder of the shared file * The share stays is in the DB and the share via the sharelink is inaccessible. (which is good) * Upgrade to owncloud 8 (8.0.2) (This step is crucial. The bug is not reproduceable without upgrading from 7 to 8. It seems like the old tokens are handled different than the newer ones) * Optional Step: Logout, Reset Browser Session, etc. * Access the share via the old share url: almost empty page, but there is a dowload button which adds a "/download" to the URL. * Upon clicking, a download.zip is downloaded which contains EVERYTHING from the owncloud directory (of the user who shared the file) * No exception is thrown and no error is logged. This will add a check whether the share is a valid one and also adds unit tests to prevent further regressions in the future. Needs to be backported to ownCloud 8. Adding a proper clean-up of the orphaned shares is out-of-scope and would probably require some kind of FK or so. Fixes https://github.com/owncloud/core/issues/15097	11 years ago
Joas Schilling	3052b9571e	Fix controller test	11 years ago
Lukas Reschke	48243a2949	Allow iframes from same domain in share view ... This is required because the PDF Viewer itself is embedded using an iframe from the same domain. The default policy is blocking this. Going on further, we have to come up with a solution in the future how to handle previews by applications, one example might be that they call their own endpoint and not the generic share page to allow applications to have full control over how to display previews. Anyways, to test this behaviour use a decent newer browser (such as Chrome 41) and share a PDF file, obviously the PDF viewer needs to be enabled as well. Without this patch publicly shared PDF files should not get previewed and an error is thrown. (if it isn't then your browser is probably not obeying our Content-Security-Policy and you might consider switching to another one ;))	11 years ago
Morris Jobke	06aef4e8b1	Revert "Updating license headers" ... This reverts commit 6a1a4880f0.	11 years ago
Joas Schilling	993d89b3cc	Extend the class so the Filesystem is cleaned up	11 years ago
Jenkins for ownCloud	6a1a4880f0	Updating license headers	11 years ago
Georg Ehrke	374ddbff55	show animated gifs on public sharing page	11 years ago
Lukas Reschke	f3ab4f3faf	Don't disclose relative directory path for single shared files of user ... The "dir" key is used within the public sharing template to indicate in which directory the user currently is when sharing a directory with subdirectories. This is needed by the JS scripts. However, when not accessing a directory then "dir" was set to the relative path of the file (from the user's home directory), meaning that for every public shared file the sharee can see the path. (For example if you share the file "foo.txt" from "finances/topsecret/" the sharee would still see the path "finances/topsecret/" from the shared HTML template) This is not the excpected behaviour and can be considered a privacy problem, this patch addresses this by setting "dir" to an empty key.	11 years ago
Vincent Cloutier	fad621140b	Added download size on public sharing	11 years ago
Lukas Reschke	988c85d292	Refactor file sharing public link handling ... fixes download issue introduced by #10755 Conflicts: apps/files_sharing/public.php	11 years ago