summaryrefslogtreecommitdiff
path: root/app/controllers
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'text-batch-1' into 'master'Dmitriy Zaporozhets2015-05-131-2/+9
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Batch 1 of text improvements Batch 1 of changes from my effort at !635 to walk through every piece of text in GitLab and see if it can be improved. This batch includes: - Improve text on error pages. - Improve Git access error messages. - Improve description of branch protection levels. - Improve OAuth signup error message. - Improve OAuth application flash messages. cc @rspeicher See merge request !642
| * Improve OAuth signup error message.Douwe Maan2015-05-131-2/+9
| |
* | Merge branch 'version-check' into 'master'Dmitriy Zaporozhets2015-05-131-0/+1
|\ \ | |/ |/| | | | | | | Version check See merge request !1509
| * Fix syntax errorDmitriy Zaporozhets2015-05-081-1/+1
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * Merge branch 'version-check' into gl-version-checkDmitriy Zaporozhets2015-05-081-0/+1
| |\ | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> Conflicts: app/controllers/admin/application_settings_controller.rb app/views/admin/application_settings/_form.html.haml db/schema.rb
| | * Add feature to disable version checkDmitriy Zaporozhets2015-03-101-1/+2
| | |
* | | Merge branch '2fa' into 'master'Dmitriy Zaporozhets2015-05-124-3/+124
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Two-factor authentication Implement's Two-factor authentication using tokens. - [X] Authentication logic - [X] Enable/disable 2FA feature - [x] Make 2-step login process if 2FA enabled - [x] Backup codes - [x] Backup code removed after being used - [x] Check backup codes for mysql db (mention mysql limitation if applied) - [x] Add tests - [x] Test if https://github.com/tinfoil/devise-two-factor#disabling-automatic-login-after-password-resets applies, and address if so - [x] Wait for fixed version of `attr_encrypted` or fork and use forked version - https://github.com/attr-encrypted/attr_encrypted/issues/155 Fixes http://feedback.gitlab.com/forums/176466-general/suggestions/4516817-implement-two-factor-authentication-2fa See merge request !474
| * | | Handle password reset for users with 2FA enabled2faRobert Speicher2015-05-111-0/+21
| | | |
| * | | Clear all 2FA-related fields when user disables the featureRobert Speicher2015-05-091-2/+7
| | | |
| * | | Prevent "You are already signed in." error message upon 2FA loginRobert Speicher2015-05-091-2/+8
| | | |
| * | | Move "invalid 2FA code" error message to the flashRobert Speicher2015-05-091-1/+1
| | | | | | | | | | | | | | | | | | | | This makes it consistent with the Invalid email/password error message from the previous step.
| * | | Refactor SessionsControllerRobert Speicher2015-05-091-22/+25
| | | | | | | | | | | | | | | | | | | | Also adds test case for providing an invalid 2FA code and then a valid one without re-entering username and password.
| * | | Fix nav and layout for TwoFactorAuthsControllerRobert Speicher2015-05-091-1/+1
| | | |
| * | | Make two-factor login work and add a feature specRobert Speicher2015-05-091-7/+24
| | | |
| * | | Fix Devise parameter sanitizer for otp_attemptRobert Speicher2015-05-091-1/+1
| | | |
| * | | Don't use hard-coded sign_in pathRobert Speicher2015-05-091-1/+1
| | | |
| * | | prepend_before_filter -> prepend_before_actionRobert Speicher2015-05-091-1/+1
| | | |
| * | | Render 2fa recovery codes instead of downloading itDmitriy Zaporozhets2015-05-091-3/+3
| | | |
| * | | Add support for backup codesDmitriy Zaporozhets2015-05-092-1/+8
| | | |
| * | | Turn 2-factor authentication into 2 steps process. Disabled 2fa UI for ldap ↵Dmitriy Zaporozhets2015-05-092-1/+25
| | | | | | | | | | | | | | | | users since it is not supported
| * | | Fix rubocop complainDmitriy Zaporozhets2015-05-091-1/+0
| | | |
| * | | Make 2 factor authentication workDmitriy Zaporozhets2015-05-091-7/+24
| | | |
| * | | Create Two-factor authentication resource for userDmitriy Zaporozhets2015-05-091-0/+22
| | | |
* | | | Merge branch 'create-merge-request' into 'master'Dmitriy Zaporozhets2015-05-111-1/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add Create Merge Request buttons to commits page and push event. Commits page: ![Screen_Shot_2015-05-09_at_22.27.09](https://gitlab.com/gitlab-org/gitlab-ce/uploads/f77c2e47f768b2fc132de6e3a2a0323e/Screen_Shot_2015-05-09_at_22.27.09.png) Branches page: ![Screen_Shot_2015-05-09_at_22.55.24](https://gitlab.com/gitlab-org/gitlab-ce/uploads/b0a6374f5c5d830ef19de9694e603c8f/Screen_Shot_2015-05-09_at_22.55.24.png) Push event: (Only for new branches, and only shown to the pushing user) ![Screen_Shot_2015-05-09_at_22.27.21](https://gitlab.com/gitlab-org/gitlab-ce/uploads/989c9cb66ee03d09b96809452749d2b8/Screen_Shot_2015-05-09_at_22.27.21.png) Push event with multiple commits and a Compare link: ![Screen_Shot_2015-05-09_at_22.28.01](https://gitlab.com/gitlab-org/gitlab-ce/uploads/a70c718e559f69177f9341c0af92707c/Screen_Shot_2015-05-09_at_22.28.01.png) See merge request !632
| * | | | Link Commits tab to current branch from Compare page.Douwe Maan2015-05-091-1/+2
| |/ / /
* | | | Don't symbolize params.params-to-symDouwe Maan2015-05-111-5/+5
|/ / /
* | | Added X-GitLab-Event header for web hooksbugagazavr2015-05-081-1/+1
|/ /
* | Merge branch 'fix-escaped-branches-in-compare' into 'master'Douwe Maan2015-05-041-2/+4
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unescape branch names in compare commit The upgrade in GitLab v7.9 to Rails v4.1.9 caused all branch names in the compare commit mode to be escaped (e.g. `/` to `%2F`). The compare mode would not always work when comparing against branch names with a forward slash. Opted to unescape the branch name rather than use a wildcard segment to prevent escaping slashes because it seems like a more sensible URL. The slashes in this case aren't really represented by a tree structure (e.g. /compare/one/two/branch..another/branch/here). * Closes #1399 * Closes https://github.com/gitlabhq/gitlabhq/issues/9105 See merge request !582
| * | Unescape branch names in compare commitStan Hu2015-04-301-2/+4
| | | | | | | | | | | | | | | Closes #1399 Closes https://github.com/gitlabhq/gitlabhq/issues/9105
* | | Merge pull request #9234 from dsander/group-milestones-by-titleDmitriy Zaporozhets2015-05-031-25/+0
|\ \ \ | | | | | | | | Group milestones by title in the dashboard and all other issue views
| * | | Removed unused ApplicationController#set_filter_values methodDominik Sander2015-05-011-25/+0
| | | | | | | | | | | | | | | | | | | | The instance variables assigned in `set_filter_values` are not used by anything anymore.
* | | | Merge branch 'change-primary-email' into 'master'Dmitriy Zaporozhets2015-05-032-7/+7
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow primary email to be set to an email that you've already added. Fixes gitlab-com/support-forum#106. When the user sets their primary email to an email that they've already added to their account, this patch makes sure that secondary email record is destroyed, and a new email record is created for the old primary email. This is based on the assumption that in this case no email was meant to be deleted, but the user simply wanted to change which of their emails is primary. See merge request !591
| * | | | Allow primary email to be set to an email that you've already added.change-primary-emailDouwe Maan2015-04-302-7/+7
| | |/ / | |/| |
* | | | Add application setting to restrict user signups to e-mail domainsStan Hu2015-05-021-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This feature was requested long ago: http://feedback.gitlab.com/forums/176466-general/suggestions/4118466-ability-to-register-only-from-ceratain-domains This MR is based off !253 but changed to use application settings and use wildcard strings to give more flexibility in pattern matching. Regexps seemed overkill and easy to get wrong. Only restrict e-mail addresses upon creation
* | | | Fix title on user page.you-get-a-title-and-you-get-a-title-and-everyone-gets-a-titleDouwe Maan2015-05-011-3/+0
| | | |
* | | | Add helpers for header title and sidebar, and move setting those from ↵Douwe Maan2015-05-0115-98/+28
| | | | | | | | | | | | | | | | controllers to layouts.
* | | | Set head panel title of help section.Douwe Maan2015-04-301-0/+7
| | | |
* | | | Fix Profiles::PasswordsController.Douwe Maan2015-04-301-1/+1
| | | |
* | | | before_action > before_filter.Douwe Maan2015-04-301-1/+1
| | | |
* | | | Remove an old determine_layout call.Douwe Maan2015-04-301-2/+0
| | | |
* | | | Add a page title to every page.Douwe Maan2015-04-3034-108/+118
| | | |
* | | | Merge branch 'show-invalid-projects-google-code-import' into 'master'Douwe Maan2015-04-301-0/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Show incompatible projects in Google Code import status Using Google Code import with a JSON file that contained only one Subversion project led to confusion over whether the system was working. Display the list of valid projects if there are any, and show a list of incompatible projects. Provide tips on how to retain issue data after conversion. Closes #1531 ## Screenshots Before: ![Screen_Shot_2015-04-29_at_12.46.41_AM](https://gitlab.com/stanhu/gitlab-ce/uploads/16ea5a99cbace48cd2f2c238b5f73f4e/Screen_Shot_2015-04-29_at_12.46.41_AM.png) After with no projects available (notice the button is hidden): ![Screen_Shot_2015-04-30_at_1.34.38_AM](https://gitlab.com/gitlab-org/gitlab-ce/uploads/43c612cdcbab181713e5764e2f38a04e/Screen_Shot_2015-04-30_at_1.34.38_AM.png) After with 1 valid and 1 incompatible project: ![Screen_Shot_2015-04-30_at_1.37.26_AM](https://gitlab.com/gitlab-org/gitlab-ce/uploads/b5bcbd304206a996932e5208ef54a071/Screen_Shot_2015-04-30_at_1.37.26_AM.png) See merge request !586
| * | | Show incompatible projects in Google Code import statusStan Hu2015-04-301-0/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | Importing a JSON file with only one Subversion project lead to confusion over whether the system was working. Provide status why these projects could not be imported directly. Closes #1531
* | | Don't allow a merge request to be merged when its title starts with "WIP".Douwe Maan2015-04-301-2/+2
|/ /
* | Draft improvements to search layoutDmitriy Zaporozhets2015-04-281-3/+6
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Merge branch 'rs-issue-2269' into 'master'Dmitriy Zaporozhets2015-04-281-12/+0
|\ \ | | | | | | | | | | | | | | | | | | | | | Remove duplicate settings for default_url_options Closes #2269 See merge request !1797
| * | Remove duplicate settings for default_url_optionsRobert Speicher2015-04-251-12/+0
| | | | | | | | | | | | Closes internal https://dev.gitlab.org/gitlab/gitlabhq/issues/2269
* | | Merge pull request #8645 from chulkilee/hipchat-api-versionDmitriy Zaporozhets2015-04-271-1/+1
|\ \ \ | | | | | | | | Add api version property on hipchat service
| * | | Add api version property on hipchat serviceChulki Lee2015-04-271-1/+1
| | | |
* | | | Revert "Added X-GitLab-Event header for web hooks"Valery Sizov2015-04-271-1/+1
|/ / / | | | | | | | | | This reverts commit 548f182814acd0f7a110e6c165c186e345901b00.