summaryrefslogtreecommitdiff
path: root/spec/requests
Commit message (Collapse)AuthorAgeFilesLines
* Do not blindly expose public project statisticsMayra Cabrera2019-06-141-2/+2
| | | | Add the missing check on GraphQL API for project statistics
* Expose comments on Noteables in GraphQLBob Van Landuyt2019-06-142-0/+114
| | | | | | | | | This exposes `Note`s on Issues & MergeRequests using a `Types::Notes::NoteableType` in GraphQL. Exposing notes on a new type can be done by implementing the `NoteableType` interface on the type. The presented object should be a `Noteable`.
* Merge branch 'expose-project-git-depth-via-api' into 'master'Kamil Trzciński2019-06-122-3/+5
|\ | | | | | | | | | | | | Expose default_git_depth via project API Closes #62908 See merge request gitlab-org/gitlab-ce!29353
| * Expose ci_default_git_depth via project APIexpose-project-git-depth-via-apiFabio Pitino2019-06-122-3/+5
| | | | | | | | | | | | | | | | | | | | Enable Get and Update of ci_default_git_depth for Project API. Renaming Project#default_git_depth to :ci_default_git_depth to give more context through the API usage. Add API documentation
* | Revert "Automatically update MR merge-ref along merge status"Oswaldo Ferreira2019-06-111-37/+24
| |
* | Merge branch 'dz-cleanup-deprecated-code' into 'master'Kamil Trzciński2019-06-111-20/+0
|\ \ | |/ |/| | | | | Remove some deprecated code See merge request gitlab-org/gitlab-ce!29222
| * Remove some deprecated codedz-cleanup-deprecated-codeDmitriy Zaporozhets2019-06-071-20/+0
| | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | Merge branch '51636-task-list-api-pderichs' into 'master'Sean McGivern2019-06-071-0/+85
|\ \ | | | | | | | | | | | | Add task count and completed count to responses of Issue and MR See merge request gitlab-org/gitlab-ce!28859
| * | Add task count and completed count to responses of Issue and MRPatrick Derichs2019-06-071-0/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add spec for task_completion_status Add test cases for task_completion_status result Extracted shared samples Add new spec file for task completion status response Fix style errors Add changelog entry Changed samples to Hashes Remove test for successful request Remove not nil expectation Add task_completion_status to api documentation for issues Add task_completion_status to api documentation for merge_requests Refactor spec so it just requests one specific item Add task_completion_status to Taskable Simplified task completion status in entities Refactor spec so it separates status code check and content check Fix spec description text and field name
* | | Merge branch '62418-project-default-git-depth' into 'master'Kamil Trzciński2019-06-071-4/+12
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Add project level git depth setting Closes #59688 See merge request gitlab-org/gitlab-ce!28919
| * | Add project level git depth settingKrasimir Angelov2019-06-061-4/+12
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce default_git_depth in project's CI/CD settings and set it to 50. Use it if there is no GIT_DEPTH variable specified. Apply this default only to newly created projects and keep it nil for old ones in order to not break pipelines that rely on non-shallow clones. default_git_depth can be updated from CI/CD Settings in the UI, must be either nil or integer between 0 and 1000 (incl). Inherit default_git_depth from the origin project when forking projects. MR pipelines are run on a MR ref (refs/merge-requests/:iid/merge) and it contains unique commit (i.e. merge commit) which doesn't exist in the other branch/tags refs. We need to add it cause otherwise it may break pipelines for old projects that have already enabled Pipelines for merge results and have git depth 0. Document new default_git_depth project CI/CD setting
* | Revert "Merge branch..."revert-86900f00Kamil Trzciński2019-06-051-2/+2
|/ | | This reverts merge request !27503
* Merge branch 'bvl-use-global-ids-graphql' into 'master'Jan Provaznik2019-06-054-4/+15
|\ | | | | | | | | | | | | Use global IDs when exposing GraphQL resources Closes #62650 See merge request gitlab-org/gitlab-ce!29080
| * Fix incorrect instances of `GraphQL::ID_TYPE`Bob Van Landuyt2019-06-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | Since the `GraphQL::ID_TYPE` usages should represent globally unique ids, this changes some fields for which this is not the case into strings. The `ID_TYPE` is a specialised, so this change should be backwards compatible. https://graphql-ruby.org/type_definitions/scalars.html
| * Expose IDs in GraphQL as a GlobalIDBob Van Landuyt2019-06-033-3/+14
| | | | | | | | | | | | | | | | | | | | This exposes all fields named `id` as GlobalIDs so they can be used across our entire GraphQL implementation. When the objects loaded are `ApplicationRecord`s. We'll use our existing batchloading to find them. Otherwise, we'll fall back to the default implementation of `GlobalID`: Calling the `.find` method on the class.
* | Merge branch 'osw-sync-merge-ref-upon-mergeability-check' into 'master'Douwe Maan2019-06-031-24/+37
|\ \ | |/ |/| | | | | | | | | Automatically update MR merge-ref along merge status Closes #58495 See merge request gitlab-org/gitlab-ce!28513
| * Automatically update MR merge-ref along merge statusOswaldo Ferreira2019-05-311-24/+37
| | | | | | | | | | | | | | | | | | | | | | | | This couples the code that transitions the `MergeRequest#merge_status` and refs/merge-requests/:iid/merge ref update. In general, instead of directly telling `MergeToRefService` to update the merge ref, we should rely on `MergeabilityCheckService` to keep both the merge status and merge ref synced. Now, if the merge_status is `can_be_merged` it means the merge-ref is also updated to the latest. We've also updated the logic to be more systematic and less user-based.
* | Merge dev.gitlab.org master into GitLab.com masterYorick Peterse2019-06-032-5/+49
|\ \
| * \ Merge branch 'security-fix_milestones_search_api_leak' into 'master'GitLab Release Tools Bot2019-06-031-4/+42
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | Resolve: Milestones leaked via search API Closes #2822 See merge request gitlab/gitlabhq!2997
| | * | Resolve: Milestones leaked via search APIFelipe Artur2019-05-201-4/+42
| | | | | | | | | | | | | | | | | | | | Fix milestone titles being leaked using search API when users cannot read milestones
| * | | Protect Gitlab::HTTP against DNS rebinding attackDouwe Maan2019-05-301-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | Gitlab::HTTP now resolves the hostname only once, verifies the IP is not blocked, and then uses the same IP to perform the actual request, while passing the original hostname in the `Host` header and SSL SNI field.
* | | | Merge branch 'ac-graphql-stats' into 'master'Dmitriy Zaporozhets2019-06-033-4/+130
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Add Namespace and ProjectStatistics to GraphQL API See merge request gitlab-org/gitlab-ce!28277
| * | | | Add Namespace and ProjectStatistics to GraphQL APIAlessio Caiazza2019-06-033-4/+130
| | |_|/ | |/| | | | | | | | | | | | | | | | | | We can query namespaces, and nested projects. Projects now exposes statistics
* | | | Abstract auto merge processesShinya Maeda2019-06-031-3/+3
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have one auto merge strategy today - Merge When Pipeline Succeeds. In order to add more strategies for Merge Train feature, we abstract the architecture to be more extensible. Removed arguments Fix spec
* | | Add optional param :start_project to allow variable commit targetsKerri Miller2019-05-311-0/+92
| | | | | | | | | | | | | | | | | | | | | | | | This extends POST#:id/repository/commits to allow the optional parameter `:start_project`, which will allow targeting other projects besides the one derived from `:id`. Resolves https://gitlab.com/gitlab-org/gitlab-ce/issues/50850
* | | Merge branch '59587-add-graphql-logging' into 'master'Sean McGivern2019-05-312-0/+73
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Add logging to GraphQL Closes #59587 See merge request gitlab-org/gitlab-ce!27885
| * | | Apply reviewer feedback59587-add-graphql-loggingcharlie ablett2019-05-312-16/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Comply doc with guidelines - Improve tests for readability and completeness - Separate out phases visually with newlines - Add `format_message` test - test readability - code and test structure/styling - static query analyzers - call `as_json` on `provided_variables` - add exception handling
| * | | Move complexity/depth to `final_value`charlieablett2019-05-301-14/+15
| | | | | | | | | | | | | | | | | | | | | | | | Tidy tests according to reviewer comments. Move complexity and depth calls from `initial_value` to `final_value` Log variables as json
| * | | Call analyzers from LoggerAnalyzercharlieablett2019-05-302-7/+15
| | | | | | | | | | | | | | | | | | | | - Add changelog file - Fix failing tests
| * | | Restructure complexity analyzercharlieablett2019-05-302-5/+23
| | | | | | | | | | | | | | | | | | | | Remove instance variables for class re-use, test individual methods, use `monotonic_time`
| * | | Implement logger analyzercharlieablett2019-05-301-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - Modify GraphqlLogger to subclass JsonLogger - Replace the single-line analyser with one that can log all the GraphQL query related information in one place. - Implement analyzer behavior with spec
| * | | New logfile for graphql queriescharlieablett2019-05-301-0/+10
| | | | | | | | | | | | | | | | | | | | Specify dedicated logfile and logger class for GraphQL queries. Move complexity analyzer to a dedicated class.
* | | | Remove legacy artifact related coderemove-legacy-artifacts-related-codeShinya Maeda2019-05-312-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | We've already migrated all the legacy artifacts to the new realm, which is ci_job_artifacts table. It's time to remove the old code base that is no longer used.
* | | | CE port of Move EE specific lines in API::ProjectsImre Farkas2019-05-301-1/+48
| | | |
* | | | CE port of Move EE specific lines in API::SettingsImre Farkas2019-05-301-0/+1
| | | |
* | | | Fix test with hard coded idHeinrich Lee Yu2019-05-301-1/+3
| | | |
* | | | Merge branch 'graphql-batch-requests-ee' into 'master'Filipa Lacerda2019-05-301-5/+24
|\ \ \ \ | |/ / / |/| | | | | | | | | | | Enable GraphQL batch requests See merge request gitlab-org/gitlab-ce!28370
| * | | Added spec for authenticating multiplex queriesPhil Hughes2019-05-291-5/+24
| | | |
* | | | Add wiki size to project statisticsPeter Marko2019-05-291-1/+2
|/ / /
* | | CE port of Move EE specific lines in API::UsersImre Farkas2019-05-281-0/+12
| | |
* | | Enables GraphQL batch requestsPhil Hughes2019-05-282-22/+102
|/ / | | | | | | | | | | | | | | | | Enabling GraphQL batch requests allows for multiple queries to be sent in 1 request reducing the amount of requests we send to the server. Responses come come back in the same order as the queries were provided.
* | Removes duplicated members from api/projects/:id/members/all51854-api-to-get-all-project-group-members-returns-duplicatesJacopo2019-05-271-0/+13
| | | | | | | | | | | | | | | | When using the members/all api the same user was returned multiple times when he was a member of the project/group and also of one of the ancestor groups. Now the member is returned only once giving priority to the membership on the project and maintaining the same behaviour of the members UI.
* | Remove the circuit breaker APIchore/remove-circuit-breaker-apiMarkus Koller2019-05-241-46/+0
| | | | | | | | | | The circuit breaker itself was removed in 11.5, this removes the corresponding API endpoints which returned empty data since then.
* | Changes RackAttack logger to use structured logsMayra Cabrera2019-05-241-0/+22
| | | | | | | | | | | | | | | | Creates a new filename to register auth logs. This change should allow SRE's queries to make better queries through logging infrastructure. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/54528
* | Merge branch 'api_masked_variables' into 'master'Bob Van Landuyt2019-05-231-1/+4
|\ \ | | | | | | | | | | | | | | | | | | API: Allow to get and set "masked" attribute for variables Closes #61754 See merge request gitlab-org/gitlab-ce!28381
| * | API: Allow to get and set "masked" attribute for variablesMathieu Parent2019-05-221-1/+4
| | |
* | | Merge branch '60250-remove-mr_push_options-flag' into 'master'Stan Hu2019-05-221-12/+0
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Remove the mr_push_options feature flag Closes #60250 See merge request gitlab-org/gitlab-ce!28278
| * | | Remove the mr_push_options feature flagLuke Duncalfe2019-05-171-12/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `Gitlab::QueryLimiting.whitelist` has been moved from being inside the feature flag conditional check to the `process_mr_push_options` `Api::Internal` helper. https://gitlab.com/gitlab-org/gitlab-ce/issues/60250
* | | | Merge branch 'graphql-tree' into 'master'Nick Thomas2019-05-222-0/+110
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Added repository files to GraphQL API See merge request gitlab-org/gitlab-ce!28325
| * | | | Added repository files to GraphQL APIPhil Hughes2019-05-222-0/+110
| | |_|/ | |/| |
View Lorry Controller Status