summaryrefslogtreecommitdiff
path: root/spec/routing
Commit message (Collapse)AuthorAgeFilesLines
* Don't add a trailing slash in group redirects40230-groups-gitlab-org-labels-json-redirects-to-groups-gitlab-org-labels-jsonSean McGivern2017-11-161-11/+17
| | | | | | | | Because we ignored the format, a request to `/groups/foo/labels.json` would redirect to `/groups/foo/-/labels/.json`. But really, it's worse than that, because unless the request contained a trailing slash, we shouldn't add one. Now, we only _keep_ a trailing slash, but don't _add_ one.
* Merge branch 'feature-change-signout-route' into 'master'Douwe Maan2017-11-081-2/+4
|\ | | | | | | | | | | | | Change Sign Out route from a DELETE to a GET Closes #39708 See merge request gitlab-org/gitlab-ce!15231
| * Updates tests to reflect sign_out route changeJoe Marty2017-11-071-2/+4
| | | | | | | | - Also remove sign_out DELETE route from read-only whitelist routes
* | Check redirecting with a querystringBob Van Landuyt2017-11-071-0/+10
| |
* | Free up some group reserved wordsBob Van Landuyt2017-11-071-17/+56
| |
* | Free up `avatar`, `group_members` and `milestones` as pathsBob Van Landuyt2017-11-072-41/+71
| |
* | Free up `labels` as a group nameBob Van Landuyt2017-11-071-0/+18
|/
* Remove Private Token from User Settings > AccountDouwe Maan2017-11-021-5/+0
|
* Allow promoting project milestones to group milestonesFelipe Artur2017-10-311-7/+12
|
* Backport EE changes to group routesbackort_routes_group_routes_from_eeValery Sizov2017-10-221-9/+3
|
* fix #3686 make tarball download url to end with extensionhaseeb2017-08-031-3/+7
|
* Change all `:empty_project` to `:project`rs-empty_project-defaultRobert Speicher2017-08-021-1/+1
|
* Remove superfluous lib: true, type: redis, service: true, models: true, ↵Rémy Coutable2017-07-271-1/+1
| | | | | | services: true, no_db: true, api: true Signed-off-by: Rémy Coutable <remy@rymai.me>
* Remove superfluous type defs in specsKeifer Furzland2017-07-271-1/+1
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Fix docker tag reference routing constraintsGrzegorz Bizon2017-07-191-0/+22
|
* Split up MergeRequestsControllerdm-merge-request-creations-controllerDouwe Maan2017-06-281-27/+46
|
* Correct RSpec/SingleLineHook cop offensesRobert Speicher2017-06-142-3/+9
|
* Implement ability to update deploy keysAlexander Randa2017-06-021-1/+3
|
* Merge branch '17848-web-hook-logging' into 'master'Dmitriy Zaporozhets2017-05-252-0/+24
|\ | | | | | | | | | | | | Implement Web Hooks calls logging Closes #17848 See merge request !11027
| * Implement web hooks loggingAlexander Randa2017-05-252-0/+24
| | | | | | | | | | | | | | | | * implemented logging of project and system web hooks * implemented UI for user area (project hooks) * implemented UI for admin area (system hooks) * implemented retry of logged webhook * NOT imeplemented log remover
* | user can reset his rss token on the account pageAlexis Reigel2017-05-241-0/+4
| |
* | Fix ambiguous routing issues by teaching router about reserved wordsDouwe Maan2017-05-231-0/+4
| |
* | Merge branch '32536-mr-widget-performance-improvements' into 'master'Fatih Acet2017-05-191-5/+0
|\ \ | | | | | | | | | | | | Load improvements related to MR widget See merge request !11518
| * | Remove routeOswaldo Ferreira2017-05-191-5/+0
| |/
* | Enable the RSpec/EmptyExampleGroup coprubocop-empty-example-groupRémy Coutable2017-05-181-10/+27
|/ | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Redirect from redirect routes to canonical routesMichael Kozono2017-05-051-3/+3
|
* Merge branch '19364-webhook-edit' into 'master' Dmitriy Zaporozhets2017-05-022-4/+16
|\ | | | | | | | | | | | | Implement ability to update hooks Closes #19364 See merge request !10816
| * Implement ability to update hooksAlexander Randa2017-04-282-4/+16
| |
* | Enable RSpec/FilePath copenable-spec-file-name-copSean McGivern2017-04-262-9/+7
|/ | | | | | - Ignore JS fixtures - Ignore qa directory - Rewrite concern specs to put concern name first
* Remove format from end of URL for URLs that take a ref or pathDouwe Maan2017-04-171-1/+1
|
* Add routing specs for environment foldersGrzegorz Bizon2017-03-271-0/+49
|
* Implement OpenID Connect identity providerMarkus Koller2017-03-071-0/+30
|
* Use native unicode emojisEric Eastwood2017-03-061-2/+1
| | | | | | | | | | | | | | | | | | | | - gl_emoji for falling back to image/css-sprite when the browser doesn't support an emoji - Markdown rendering (Banzai filter) - Autocomplete - Award emoji menu - Perceived perf - Immediate response because we now build client-side - Update `digests.json` generation in gemojione rake task to be more useful and include `unicodeVersion` MR: !9437 See issues - #26371 - #27250 - #22474
* Remove query parameters from notes polling endpoint to make caching easierAdam Niedzielski2017-03-011-4/+14
|
* Merge branch 'master' into jej-pages-picked-from-eeJames Edwards-Jones2017-02-031-4/+4
|\
| * replace `find_with_namespace` with `find_by_full_path`Adam Pahlevi2017-02-031-4/+4
| | | | | | | | add complete changelog for !8949
* | Fix 500 error while navigating to the `pages_domains` 'show' page.Timothy Andrew2017-01-311-7/+30
|/ | | | | | | | | | | | | | | | | | | | | | | | | | | | ================== = Implementation = ================== 1. The path of the page is of the form 'group/project/pages/domains/<domain_name>' 2. Rails looks at `params[:id]` (which should be the domain name), and finds the relevant model record. 3. Given a domain like `foo.bar`, Rails sets `params[:id]` to `foo` (should be `foo.bar`), and sets `params[:format]` to `bar` 4. This commit fixes the issue by adding a route constraint, so that `params[:id]` is set to the entire `foo.bar` domain name. ========= = Tests = ========= 1. Add controller specs for the `PagesDomainController`. These are slightly orthogonal to this bug fix (they don't fail when this bug is present), but should be present nonetheless. 2. Add routing specs that catch this bug (by asserting that the `id` param is passed as expected when it contains a domain name). 3. Modify the 'RESTful project resources' routing spec shared example to accomodate controllers where the controller path (such as `pages/domains`) is different from the controller name (such as `pages_domains`).
* Fix 500 error when visit group from admin area if group name contains dotdz-improve-admin-group-routingDmitriy Zaporozhets2016-12-281-2/+8
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Modify GithubImport to support GiteaRémy Coutable2016-12-191-1/+0
| | | | | | | | The reason is that Gitea plan to be GitHub-compatible so it makes sense to just modify GitHubImport a bit for now, and hopefully we can change it to GitHubishImport once Gitea is 100%-compatible. Signed-off-by: Rémy Coutable <remy@rymai.me>
* Rename Gogs to Gitea, DRY the controller and improve viewsRémy Coutable2016-12-191-0/+166
| | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* Changed autocomplete_sources into an action that returns a single 'at' type ↵18435-autocomplete-is-not-performantLuke Bennett2016-12-151-4/+15
| | | | | | | | | | | | | | | | | | | | | | | | of sources at a time Finished up autocomplete_sources action and added frontend to fetch data only when its needed Added wait_for_ajax to specs Fixed builds and improved the setup/destroy lifecycle Changed global namespace and DRYed up loading logic Added safety for accidentally loading data twice Removed destroy as its not necessary and is messing with click events from a blur race condition Created AutocompleteSourcesController and updated routes Fixed @undefined from tabbing before load ends Disable tabSelectsMatch until we have loaded data Review changes
* Add support for nested groups to admin routingdz-fix-admin-routesDmitriy Zaporozhets2016-12-111-1/+13
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Add nested groups support to the routingdz-allow-nested-group-routingDmitriy Zaporozhets2016-11-232-510/+540
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix 500 error when group name ends with gitDmitriy Zaporozhets2016-11-211-1/+1
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix 404 on some group pages when name contains dotdz-fix-group-name-dotDmitriy Zaporozhets2016-11-211-5/+13
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Fix routing spec for group controllerdz-refactor-contraintsDmitriy Zaporozhets2016-11-081-2/+2
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'dz-rename-user-routes' into 'master'Robert Speicher2016-10-171-6/+6
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | Rename users routing from /u/:username to /users/:username for consistency with other routes Renames /u/:username to /users/:username To follow consistency with other routes (like groups) and UsersController name. Now when you can use `/:username` for accessing user page there is no need in shortcut like `/u/` See merge request !6851
| * Update users routing specDmitriy Zaporozhets2016-10-131-6/+6
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Fix 404 when group path has dot in the namedz-fix-group-with-dotDmitriy Zaporozhets2016-10-161-0/+6
|/ | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge branch 'atom-routes' into 'master' Robert Speicher2016-10-111-1/+1
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow browsing branches that end with '.atom' ## What does this MR do? 1. Simplify the regex capture in the routing for the CommitsController to not exclude the '.atom' suffix. That's a perfectly valid git branch name, so we shouldn't blow up if we get it. 2. Because Rails now can't automatically detect the request format, add some code to do so in `ExtractPath` when there is no path. This means that, given branches 'foo' and 'foo.atom', the Atom feed for the former is unroutable. To fix this: don't do that! Give the branches different names! ## Why was this MR needed? Creating a branch or tag name ending in '.atom' would cause some 500s on that repo. ## What are the relevant issue numbers? Closes #21955. Related to !5994. See merge request !6750
View Lorry Controller Status