summaryrefslogtreecommitdiffstats
path: root/core/Controller
Commit message (Collapse)AuthorAgeFilesLines
* Clean pending 2FA authentication on password resetRoeland Jago Douma2019-01-301-2/+8
| | | | | | | | | | | When a password is reste we should make sure that all users are properly logged in. Pending states should be cleared. For example a session where the 2FA code is not entered yet should be cleared. The token is now removed so the session will be killed the next time this is checked (within 5 minutes). Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Honor remember_login_cookie_lifetimeRoeland Jago Douma2019-01-231-1/+8
| | | | | | | | | | | | If the remember_login_cookie_lifetime is set to 0 this means we do not want to use remember me at all. In that case we should also not creatae a remember me cookie and should create a proper temp token. Further this specifies that is not 0 the remember me time should always be larger than the session timeout. Because else the behavior is not really defined. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fix template paramterRoeland Jago Douma2019-01-221-1/+1
| | | | | | Else we get shown an error page instead of the correct 403. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Fix loginflow with apptoken enter on iOSRoeland Jago Douma2019-01-041-0/+12
| | | | | | | It seems iOS doesn't like us to change the location. So now we submit it to the server that geneartes the redirect. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Use only app path to get the icon on the svg apiJohn Molakvoæ (skjnldsv)2018-12-211-3/+4
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix SAML Client login flow on Apple devicesRoeland Jago Douma2018-12-171-0/+1
| | | | | | | | | | | Because the redirect from the SAML/SSO endpoint is a POST the lax/strict cookies are not properly send. Note that it is not strictly requried on this endpoint as we do not need the remember me data. Only the real session info is enough. The endpoint is also already protected by a state token. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Respect the disabled setting for lost_password_linkRoeland Jago Douma2018-11-201-1/+3
| | | | | | | | Fixes #11146 As documented when it is set to disabled the user can't request a lost password. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Use the proper server for the apptoken flow loginRoeland Jago Douma2018-11-061-21/+23
| | | | | | | | | | | | | If a user can't authenticate normally (because they have 2FA that is not available on their devices for example). The redirect that is generated should be of the proper format. This means 1. Include the protocol 2. Include the possible subfolder Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Remove cookies from Clear-Site-Data HeaderPatrick Conrad2018-10-241-1/+1
| | | | | | | | | | | | | | In https://github.com/nextcloud/server/commit/2f87fb6b456fd109c90a5093c31b7a3f62a32040 this header was introduced. The referenced documentation says: > When delivered with a response from https://example.com/clear, the following header will cause cookies associated with the origin https://example.com to be cleared, as well as cookies on any origin in the same registered domain (e.g. https://www.example.com/ and https://more.subdomains.example.com/). This also applies if `https://nextcloud.example.com/` sends the `Clear-Site-Data: "cookies"` header. This is not the behavior we want at this point! So I removed the deletion of cookies from the header. This has no effect on the logout process as this header is supported only recently and the logout works in old browsers as well. Signed-off-by: Patrick Conrad <conrad@iza.org> (cherry picked from commit 1806baaeafa284808cceb1a38ea2e1a9189d0407)
* Do not invalidate main token on OAuthRoeland Jago Douma2018-09-061-3/+3
| | | | | | | | | | | | | | | | Fixes #10584 We deleted the main token when using the login flow else mutliple tokens would show up for a single user. However in the case of OAuth this is perfectly fine as the authentication happens really in your browser: 1. You are already logged in, no need to log you out 2. You are not logged in yet, but since you log in into the exact same browser the expected behavior is to stay logged in. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Change password expiration time from 12h to 7dMorris Jobke2018-08-311-1/+1
| | | | | | We use the same logic for creating accounts without a password and there the 12h is a bit short. Users don't expect that the signup link needs to be clicked within 12h - 7d should be a more expected behavior. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #10670 from nextcloud/fix/login-throttle-username-lengthMorris Jobke2018-08-241-1/+1
|\ | | | | Fix max length requirements for the throttler metadata
| * Fix max length requirements for the throttler metadataChristoph Wurst2018-08-131-1/+1
| | | | | | | | | | | | | | | | | | If a failed login is logged, we save the username as metadata in the bruteforce throttler. To prevent database error due to very long strings, this truncates the username at 64 bytes in the assumption that no real username is longer than that.long strings, Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | only warn about data lose on password reset if per-user keys are usedBjoern Schiessle2018-08-151-1/+10
|/ | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* Rename providerset method to get primary providersChristoph Wurst2018-08-081-1/+1
| | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Fix login redirection if only one 2FA provider is activeChristoph Wurst2018-08-081-1/+1
| | | | | | Fixes https://github.com/nextcloud/server/issues/10500. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Make the info available if the avatar was uploaded or generatedJoas Schilling2018-08-011-5/+6
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #10391 from ↵Morris Jobke2018-07-251-1/+2
|\ | | | | | | | | nextcloud/bugfix/10389/fix-app-navigation-controller Fix app navigation controller to return an array
| * Fix app navigation controller to return an arrayJulius Härtl2018-07-251-1/+2
| | | | | | | | | | | | This is required to not break compatibility with existing consumers of that endpoint like the apps management or the client Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Allow to access SVGs as guestJoas Schilling2018-07-251-3/+3
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Implements handling for deactivated usersMichael Weimann2018-07-211-25/+84
| | | | Signed-off-by: Michael Weimann <mail@michael-weimann.eu>
* Php code bumpJohn Molakvoæ (skjnldsv)2018-07-191-5/+13
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix settings icon fetchingJulius Härtl2018-07-191-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Fixed svg generator regex && added missing iconsJohn Molakvoæ (skjnldsv)2018-07-191-1/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fixed icons detection and cachingJohn Molakvoæ (skjnldsv)2018-07-191-10/+4
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Mixin scss icon apiJohn Molakvoæ (skjnldsv)2018-07-191-1/+7
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* use built in response cacheFor functionJohn Molakvoæ (skjnldsv)2018-07-191-1/+1
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Svg icon api sass function and upgrade of all stylesJohn Molakvoæ (skjnldsv)2018-07-191-2/+10
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Svg from appJohn Molakvoæ (skjnldsv)2018-07-191-1/+32
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Svg color apiJohn Molakvoæ (skjnldsv)2018-07-192-2/+99
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* wire the frontendArthur Schiwon2018-07-051-2/+11
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Backend work to provide NC whats New info to usersArthur Schiwon2018-07-051-0/+117
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Merge pull request #9632 from nextcloud/enhancement/stateful-2fa-providersMorris Jobke2018-06-252-5/+26
|\ | | | | Stateful 2fa providers
| * Make 2FA providers statefulChristoph Wurst2018-06-202-5/+26
| | | | | | | | | | | | | | | | | | | | | | This adds persistence to the Nextcloud server 2FA logic so that the server knows which 2FA providers are enabled for a specific user at any time, even when the provider is not available. The `IStatefulProvider` interface was added as tagging interface for providers that are compatible with this new API. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | prefill userid for login after password resetRobin Appelman2018-06-211-3/+4
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Removed debug stackJohn Molakvoæ (skjnldsv)2018-06-151-1/+0
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Avatar imagick bumpJohn Molakvoæ (skjnldsv)2018-06-131-48/+14
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Controller tests fixesJohn Molakvoæ (skjnldsv)2018-06-131-6/+16
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Return Svg avatarsJohn Molakvoæ (skjnldsv)2018-06-131-9/+37
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Fix usageRoeland Jago Douma2018-06-042-18/+0
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #9531 from remyj38/masterMorris Jobke2018-05-221-0/+3
|\ | | | | Fix translation bug on lost password page
| * Fix translation bug on lost password pageRémy Jacquin2018-05-201-0/+3
| | | | | | | | | | | | Fix nextcloud/password_policy#26 Signed-off-by: Rémy Jacquin <remy@remyj.fr>
* | Don't use special chars to avoid confusionRoeland Jago Douma2018-05-221-1/+1
|/ | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* move log constants to ILoggerArthur Schiwon2018-04-261-1/+2
| | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Use proper typesRoeland Jago Douma2018-04-111-4/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Avoid to leak a user ID that is not a string to reach a user backendMorris Jobke2018-04-111-0/+4
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #9115 from nextcloud/feature/noid/login_flow_wordingRoeland Jago Douma2018-04-091-0/+38
|\ | | | | Improve login flow
| * Improve login flowRoeland Jago Douma2018-04-081-0/+38
| | | | | | | | | | | | | | * Add page explaining you are about to grant access * Show grant access page after login Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Remove unused parameterRoeland Jago Douma2018-04-061-2/+1
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Fix testsRoeland Jago Douma2018-04-061-1/+2
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>