summaryrefslogtreecommitdiffstats
path: root/lib/private/appframework
Commit message (Collapse)AuthorAgeFilesLines
* make resolve public to avoid boiler plate codeBernhard Posselt2015-09-131-1/+1
| | | | add resolve to public interface
* Properly return 304Roeland Jago Douma2015-09-011-1/+1
| | | | | | | | The ETag set in the IF_NONE_MODIFIED header is wraped in quotes ("). However the ETag that is set in response is not (yet). Also we need to cast the ETag to a string. * Added unit test
* Decode request content only on getContentRobin McCorkell2015-08-311-34/+49
|
* Merge pull request #17662 from owncloud/locking-dbThomas Müller2015-08-261-0/+9
|\ | | | | Database backend for locking
| * add method to check if we're inside a transactionRobin Appelman2015-08-101-0/+9
| |
* | Add mitigation against BREACHLukas Reschke2015-08-141-1/+21
| | | | | | | | | | | | | | | | | | | | | | | | While BREACH requires the following three factors to be effectively exploitable we should add another mitigation: 1. Application must support HTTP compression 2. Response most reflect user-controlled input 3. Response should contain sensitive data Especially part 2 is with ownCloud not really given since user-input is usually only echoed if a CSRF token has been passed. To reduce the risk even further it is however sensible to encrypt the CSRF token with a shared secret. Since this will change on every request an attack such as BREACH is not feasible anymore against the CSRF token at least.
* | Merge pull request #17982 from owncloud/appframework-sanitize-nameThomas Müller2015-08-121-0/+10
|\ \ | | | | | | Sanitize class names before registerService/query
| * | Sanitize class names before registerService/queryRobin McCorkell2015-07-301-0/+10
| |/ | | | | | | | | Leading backslashes are removed, so a `registerService('\\OC\\Foo')` can still be resolved with `query('OC\\Foo')`.
* | Merge pull request #17852 from owncloud/register-alias-factoryRobin McCorkell2015-08-112-1/+2
|\ \ | | | | | | Add test for factories
| * | add test for factoriesBernhard Posselt2015-07-252-1/+2
| |/ | | | | | | | | | | | | | | | | | | use ref for factory test use a factory for registerAlias Ensure we construct SimpleContainer Use single instance of DIContainer in routing tests
* | Set default `forwarded_for_headers` to 'HTTP_X_FORWARDED_FOR'Robin McCorkell2015-08-101-1/+4
| |
* | Use DIRoeland Jago Douma2015-08-101-0/+14
| | | | | | | | | | | | | | | | * Register OCP\Capability\IManager at DIContainer * Add register capabilities to appframework * Register capabilities in DI way * Make unit test pass again * Remove CapabiltiesManager from OCP
* | Add \OCP\ISession to AppFrameworkRobin McCorkell2015-08-071-0/+4
| |
* | Remove "use" statementLukas Reschke2015-08-051-2/+1
| | | | | | | | Ref https://bugs.php.net/bug.php?id=66773
* | Use StringUtils::equals on CSRF token and add unit testsLukas Reschke2015-08-041-5/+4
|/
* Merge pull request #17755 from owncloud/alias-container-aliveThomas Müller2015-07-242-50/+41
|\ | | | | Add registerAlias method to shortcut interface registration #17714
| * add registerAlias method to shorcut interface registrationBernhard Posselt2015-07-182-50/+41
| | | | | | | | | | | | | | | | remove unused import add since tag fix typo
* | Add a log message when the Doctrine Query Builder is retrievedJoas Schilling2015-07-211-9/+0
| |
* | Add unit tests and automatic quotingJoas Schilling2015-07-211-2/+2
| |
* | Add QueryBuilder, ExpressionBuilder and CompositeExpression wrappersJoas Schilling2015-07-211-0/+18
| |
* | Return proper status code in case of a CORS exceptionLukas Reschke2015-07-202-11/+40
|/ | | | | | When returning a 500 statuscode external applications may interpret this as an error instead of handling this more gracefully. This will now make return a 401 thus. Fixes https://github.com/owncloud/core/issues/17742
* Fixing 'Undefined index: REMOTE_ADDR' - fixes #17460Thomas Müller2015-07-161-1/+2
|
* Merge pull request #17078 from owncloud/fix-initial-server-hostMorris Jobke2015-07-011-6/+5
|\ | | | | Fix undefined offset
| * Fix undefined offsetLukas Reschke2015-06-221-6/+5
| | | | | | | | There are cases where no trusted host is specified such as when installing the instance, this lead to an undefined offset warning in the log right after installing. (when another domain than localhost or 127.0.0.1 was used)
* | update license headers and authorsMorris Jobke2015-06-254-1/+4
| |
* | Allow multiple whitespace in type hints in AppFrameworkRobin McCorkell2015-06-201-1/+1
|/ | | | | | Type hints such as `@param bool $doSomething` will now correctly get parsed, allowing for alignment of docblock parameters if the app developer so wishes.
* disallow cookie auth for cors requestsBernhard Posselt2015-05-222-7/+47
| | | | | | | | | | | | | | | | testing ... fixes fix test add php doc fix small mistake add another phpdoc remove not working cors annotations from files app
* Scrutinizer Auto-FixesScrutinizer Auto-Fixer2015-05-191-1/+0
| | | | This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com
* Check return value of OC_App::getAppPath() and verify info.xml exists.Andreas Fischer2015-05-061-11/+14
|
* add aliases to pascal case constructor paramters to make it possible to auto ↵Bernhard Posselt2015-04-291-0/+11
| | | | assemble controllers
* Merge pull request #15735 from owncloud/fix-visibilityMorris Jobke2015-04-202-3/+3
|\ | | | | Fix visibility of interfaces in \OCP
| * Fix visibility of public API methodsJoas Schilling2015-04-202-3/+3
| |
* | Remove unused variablesMorris Jobke2015-04-181-2/+1
|/ | | | | * should make scrutinizer a lot more happy * reduces maybe memory footprint
* Update license headersJenkins for ownCloud2015-03-2618-208/+275
|
* Also add a note that clob does not work in the compare arrayJoas Schilling2015-03-161-0/+1
|
* Fix doc blocks of insertIfNotExists() methodJoas Schilling2015-03-111-16/+8
|
* Allow specifying the compare-array for insertIfNotExists()Joas Schilling2015-03-091-2/+2
|
* Let users configure security headers in their WebserverLukas Reschke2015-03-022-2/+2
| | | | | | | | | | Doing this in the PHP code is not the right approach for multiple reasons: 1. A bug in the PHP code prevents them from being added to the response. 2. They are only added when something is served via PHP and not in other cases (that makes for example the newest IE UXSS which is not yet patched by Microsoft exploitable on ownCloud) 3. Some headers such as the Strict-Transport-Security might require custom modifications by administrators. This was not possible before and lead to buggy situations. This pull request moves those headers out of the PHP code and adds a security check to the admin settings performed via JS.
* AppFramework StreamResponseBernhard Posselt2015-02-274-17/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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"Morris Jobke2015-02-2617-245/+230
| | | | This reverts commit 6a1a4880f0d556fb090f19a5019fec31916f5c36.
* owncloud -> ownCloudThomas Müller2015-02-251-2/+2
|
* migrate to IDBConnectionBernhard Posselt2015-02-251-5/+182
|
* Merge pull request #14458 from owncloud/revive/11157Thomas Müller2015-02-251-3/+9
|\ | | | | Get the real protocol behind several proxies
| * Use [0] instead of current as HHVM might have problems with thatLukas Reschke2015-02-241-1/+1
| |
| * Get the real protocol behind several proxiesLukas Reschke2015-02-241-3/+9
| | | | | | | | | | | | X-Forwarded-Proto contains a list of protocols if ownCloud is behind multiple reverse proxies. This is a revival of https://github.com/owncloud/core/pull/11157 using the new IRequest public API.
* | Merge pull request #14151 from owncloud/update-sabre2.1Thomas Müller2015-02-251-2/+2
|\ \ | |/ |/| Sabre Update to 2.1
| * Sabre Update to 2.1Vincent Petry2015-02-231-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - VObject fixes for Sabre\VObject 3.3 - Remove VObject property workarounds - Added prefetching for tags in sabre tags plugin - Moved oc_properties logic to separate PropertyStorage backend (WIP) - Fixed Sabre connector namespaces - Improved files plugin to handle props on-demand - Moved allowed props from server class to files plugin - Fixed tags caching for files that are known to have no tags (less queries) - Added/fixed unit tests for Sabre FilesPlugin, TagsPlugin - Replace OC\Connector\Sabre\Request with direct call to httpRequest->setUrl() - Fix exception detection in DAV client when using Sabre\DAV\Client - Added setETag() on Node instead of using the static FileSystem - Also preload tags/props when depth is infinity
* | properly initialize OC::$WEBROOT and host nameThomas Müller2015-02-231-1/+1
|/
* Merge pull request #14427 from owncloud/fix-some-php-docsBernhard Posselt2015-02-231-3/+4
|\ | | | | Fix some PHPDocs
| * Fix some PHPDocsLukas Reschke2015-02-231-3/+4
| |