| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
|\
| |
| | |
Add `getNonExistingName()` to the node api
|
| | |
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|\
| |
| | |
Preview provider registration in manager
|
| |
| |
| |
| | |
We shall add a factory for that in the future
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| | |
value instead of index
|
|\ \
| | |
| | | |
Fix backwards compatibility for mapper execute method
|
| | | |
|
| | | |
|
|\ \ \
| |/ /
|/| | |
pass mountpoint to storage wrapper callback
|
| | | |
|
| | | |
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | |
| | | | |
owncloud/issue/14881-closures-for-navigation-entries
Issue/14881 closures for navigation entries
|
| | |/
| |/|
| | |
| | |
| | | |
This speeds up all pages that don't use the navigation by 0.04sec per app,
because we don't need to create the routing anymore, unless we really need to.
|
|\ \ \
| | | |
| | | | |
Migrate OC_Mail to SwiftMailer
|
| | | | |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Replaces the OC_Mail and phpmailer with SwiftMail allowing us to mock it properly.
Fixes the unit test execution on master on local machines and https://github.com/owncloud/core/issues/12014
Conflicts:
3rdparty
lib/private/server.php
lib/public/iservercontainer.php
tests/lib/mail.php
tests/settings/controller/mailsettingscontrollertest.php
Conflicts:
3rdparty
lib/private/mail.php
lib/private/server.php
lib/public/iservercontainer.php
settings/ajax/lostpassword.php
settings/application.php
|
| |/ /
|/| | |
|
|\ \ \
| | | |
| | | | |
Allow specifying the compare-array for insertIfNotExists()
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
real exception
|
| | | | |
|
|\ \ \ \
| |_|/ /
|/| | | |
verify the file name length not to exceed 255 characters + verify path d...
|
| | | |
| | | |
| | | |
| | | | |
during rename
|
|/ / /
| | |
| | |
| | | |
If the user does not exist this returns null and can lead to nasty bugs since the IDE is not indicating this...
|
| |/
|/|
| |
| |
| |
| |
| |
| | |
* Original avatarcontroller migrated to the appframework
* Added DataDisplayResponse that show data inline in the browser (used
to retrun the image)
* Removed some unneeded code
* Added unit tests for the avatarcontroller
|
|\ \
| | |
| | | |
adding storage specific filename verification
|
| | | |
|
| | | |
|
| |/ |
|
|\ \
| |/
|/| |
Add support for 'child-src' directive
|
| |
| |
| |
| |
| |
| | |
This is required when working with stuff such as PDF.js in the files_pdfviewer application. Opt-in only.
Master change only because the stable CSP policies has a failback that allows nearly anything :see_no_evil:
|
| |
| |
| |
| | |
That apparently went lost when changing the signature.
|
|/ |
|
|\
| |
| | |
AppFramework StreamResponse
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
First stab at the StreamResponse, see #12988
The idea is to use an interface ICallbackResponse (I'm not 100% happy with the name yet, suggestions?) that allow the response to output things in its own way, for instance stream the file using readfile
Unittests are atm lacking, plan is to
check if a mock of ICallbackResponse will be used by calling its callback (also unhappy with this name) method
Usage is:
$response = new StreamResponse('path/to/file');
rename io to output, add additional methods and handle error and not modified cases when using StreamResponse
fix indention and uppercasing, also handle forbidden cases
fix indention
fix indention
no forbidden, figuring out if a file is really readable is too complicated to get to work across OSes and streams
remove useless import
remove useless import
fix intendation
|
|/ |
|
|\
| |
| | |
Revert "Updating license headers"
|
| |
| |
| |
| | |
This reverts commit 6a1a4880f0d556fb090f19a5019fec31916f5c36.
|
|\ \
| |/
|/| |
Add async command system to handle asynchronous operations
|