Stan Hu
19f478e00d
Fix Error 500 in API when accessing a group that has an avatar
...
Closes #2340
2015-08-24 07:46:06 -07:00
Stan Hu
ed1d4fa477
Remove user OAuth tokens stored in database for Bitbucket, GitHub, and GitLab
...
and request them each session. Pass these tokens to the project import data.
This prevents the need to encrypt these tokens and clear them in case they
expire or get revoked.
For example, if you deleted and re-created OAuth2 keys for Bitbucket, you would get
an Error 500 with no way to recover:
```
Started GET "/import/bitbucket/status" for x.x.x.x at 2015-08-07 05:24:10 +0000
Processing by Import::BitbucketController#status as HTML
Completed 500 Internal Server Error in 607ms (ActiveRecord: 2.3ms)
NameError (uninitialized constant Import::BitbucketController::Unauthorized):
app/controllers/import/bitbucket_controller.rb:77:in `rescue in go_to_bitbucket_for_permissions'
app/controllers/import/bitbucket_controller.rb:74:in `go_to_bitbucket_for_permissions'
app/controllers/import/bitbucket_controller.rb:86:in `bitbucket_unauthorized'
```
Closes #1871
2015-08-23 09:23:44 -07:00
Maël Valais
a9e409179b
Create cross-reference for closing references on commits pushed to non-default branches.
...
I also revamped the tests on "closing reference commits" (= "Fixes #xxxx" for example).
Now, there are two different contexts:
- when the commits with "closing reference" are pushed to the default branch,
- when the commits with "closing reference" are pushed to a non-default branch.
Closes gitlab-org/gitlab-ce#2190 .
2015-08-22 16:49:09 +02:00
Robert Speicher
f0bdf7f810
Merge branch 'reply-by-email' into 'master'
...
Reply by email
Fixes #1360 .
It's far from done, but _it works_.
See merge request !1173
2015-08-22 00:00:08 +00:00
Douwe Maan
15fc7bd613
No HTML-only email please
2015-08-21 16:09:55 -07:00
Douwe Maan
0daa21ed8c
Merge branch 'fix-email-autocomplete' into 'master'
...
Fix bug preventing invite by e-mail
This was broken in 70f5291808 as a result of introducing the ability to filter by current user.
Closes #2320
See merge request !1181
2015-08-21 22:25:59 +00:00
Stan Hu
3715a0f29b
Fix bug preventing invite by e-mail
...
This was broken in 70f5291808 as a result of introducing the ability to filter by current user.
Closes #2320
2015-08-21 13:47:56 -07:00
Douwe Maan
1cfc4af31e
Merge branch 'rs-dev-issue-2550' into 'master'
...
Remove trailing HTML entities from non-Rinku autolinks as well.
Addresses internal https://dev.gitlab.org/gitlab/gitlabhq/issues/2550
See merge request !1179
2015-08-21 18:48:18 +00:00
Douwe Maan
69708dab9f
Block blocked users from replying to threads by email.
2015-08-21 10:14:45 -07:00
Dmitriy Zaporozhets
d43d7ffd6f
Merge branch 'hide-recent-push' into 'master'
...
Only show recent push event if the branch still exists or a recent merge request has not been created
Closes #2277
See merge request !1167
2015-08-21 14:54:53 +00:00
Dmitriy Zaporozhets
59525d6b28
Merge branch 'comment-case-insensetive-search' into 'master'
...
Case-insensitive search for comments
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com >
See merge request !1174
2015-08-21 14:53:36 +00:00
Robert Speicher
747fe7520b
Remove trailing HTML entities from non-Rinku autolinks as well.
2015-08-20 18:25:16 -07:00
Douwe Maan
afb765ad9e
Fix markdown specs again. Apparently development and test behave differently.
2015-08-20 15:41:34 -07:00
Douwe Maan
99ef8c8159
Fix markdown specs.
2015-08-20 15:37:43 -07:00
Douwe Maan
48e25a019a
Add stub_reply_by_email_setting helper.
2015-08-20 13:21:22 -07:00
Douwe Maan
6110e175dc
Use heredocs.
2015-08-20 13:10:55 -07:00
Douwe Maan
54b04f1c5b
Add fixture_file helper.
2015-08-20 12:41:47 -07:00
Douwe Maan
ac268674ed
Update spec.
2015-08-20 12:34:45 -07:00
Douwe Maan
e1eb09dd0e
Remove more unused fixtures.
2015-08-20 12:33:33 -07:00
Douwe Maan
2b5a2b8f39
Removed unused fixtures.
2015-08-20 12:29:03 -07:00
Douwe Maan
3d51a6d435
Merge branch 'master' into reply-by-email
2015-08-20 12:19:19 -07:00
Douwe Maan
8ec5fb138d
Test Gitlab::Email::Receiver.
2015-08-20 12:17:59 -07:00
Douwe Maan
e44936f3ed
Test EmailReceiverWorker.
2015-08-20 11:47:09 -07:00
Douwe Maan
991c9f6fda
Test Email::AttachmentUploader.
2015-08-20 11:33:35 -07:00
Douwe Maan
e9972efc2f
Extract ReplyParser and AttachmentUploader from Receiver.
2015-08-20 11:05:06 -07:00
Stan Hu
98eb89be5d
Only show recent push event if the branch still exists or a recent merge request has not been created
...
Closes #2277
2015-08-20 01:38:15 -07:00
Stan Hu
0a5b005da1
Fix blame view line groupings
...
Closes #2305
2015-08-20 01:27:29 -07:00
Douwe Maan
83081f1673
Start on tests.
2015-08-19 18:00:13 -07:00
Douwe Maan
41fdd20c74
Test Gitlab::ReplyByEmail.
2015-08-19 16:05:58 -07:00
Dmitriy Zaporozhets
b7e6ae03dc
Case-insensetive search for comments
...
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com >
2015-08-19 15:59:48 +02:00
Dmitriy Zaporozhets
72a425fa66
Merge branch 'remove-satellites'
...
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com >
2015-08-19 09:52:39 +02:00
Robert Speicher
204914983a
Merge branch 'fix-authhash-infinite-loop' into 'master'
...
Fix infinite loop when SAML was incorrectly configured.
See merge request !1170
2015-08-18 15:56:30 +00:00
Dmitriy Zaporozhets
798f2fe19f
Merge branch 'mr-source-branch-clear-cache' into 'master'
...
Clear cache to prevent listing deleted branches after MR removes source branch
Closes #2272
See merge request !1160
2015-08-18 13:16:46 +00:00
Douwe Maan
262c6f11fe
Merge branch 'rs-remove-gfm_with_options' into 'master'
...
Remove the `gfm_with_options` helper
It was redundant because `gfm` also took options.
See merge request !1152
2015-08-17 23:42:29 +00:00
Douwe Maan
c16b1651f5
Fix infinite loop when SAML was incorrectly configured.
2015-08-17 16:39:22 -07:00
Douwe Maan
c9920c422d
Merge branch 'fix-backslashes-inline-diff' into 'master'
...
Fix bug where backslashes in inline diffs could be dropped
This MR fixes a bug in inline diff generation causing backslashes to be dropped. For example, the input:
```
input.to_s.sub(/[\r\n].+/,'').sub(/\\[rn].+/, '').strip
```
The second backslash is dropped in the second `sub` statement:

With this fix, it looks like:

Closes #2253
See merge request !1143
2015-08-17 23:33:36 +00:00
Dmitriy Zaporozhets
4a0fa27017
Merge branch 'master' into remove-satellites
...
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com >
2015-08-17 11:41:53 +02:00
Stan Hu
d416a9d624
Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce
2015-08-14 08:51:49 -07:00
Boyan Tabakov
88ab815b38
Update Flowdock integration to support new Flowdock API
...
Requires that users create a Git source in their flows and update the configured token to the Git source token. The old flow tokens can't be used.
2015-08-14 16:45:23 +02:00
Stan Hu
6e808fc222
Clear cache to prevent listing deleted branches after MR removes source branch
...
Closes #2272
2015-08-14 07:09:25 -07:00
Dmitriy Zaporozhets
4e4866f255
Refactor pre/post receive commit services into one class
2015-08-14 16:04:17 +02:00
Dmitriy Zaporozhets
9649f9387a
Fix tests for web editor
2015-08-14 15:08:31 +02:00
Dmitriy Zaporozhets
9011a32234
Disable pre-receive check in test env
...
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com >
2015-08-14 13:27:51 +02:00
Dmitriy Zaporozhets
3483024d93
Merge branch 'disable-internal-tracker-when-external-in-use' into 'master'
...
Disable internal issue tracker if external tracker in use
### What does this MR do?
This MR disables routes to the internal issue tracker if an external issue tracker (e.g. Redmine) is in use. A 404 page is returned if a user attempts to access any of the issue routes.
### Why was this MR needed?
Enabling the external issue tracker would still keep the internal one active, which would lead to odd behavior where users could create issues but not close them.
### What are the relevant issue numbers?
Closes #2006
See merge request !993
2015-08-14 11:01:43 +00:00
Dmitriy Zaporozhets
00a33d67aa
Merge branch 'improve-hipchat-service-test' into 'master'
...
Provide more feedback what went wrong if HipChat service failed test
This MR adds support for displaying the error message during HipChat service test. Before an Error 500 would be displayed with no helpful remarks.
Screenshot:

Issue gitlab-com/support-forum#213
See merge request !1144
2015-08-13 11:43:44 +00:00
Robert Speicher
907860ed3d
Remove the gfm_with_options helper
...
It was redundant because `gfm` also took options.
2015-08-12 23:32:16 -04:00
Ben Boeckel
013260e057
markup_helper: detect .mkd and .mkdn as markdown
2015-08-12 15:29:15 -04:00
Stan Hu
2379057002
Provide more feedback what went wrong if HipChat service failed test
...
Issue gitlab-com/support-forum#213
2015-08-12 07:31:25 -07:00
Stan Hu
c36adb98aa
Fix bug where backslashes in inline diffs could be dropped
...
Closes #2253
2015-08-11 18:39:27 -07:00
Florent (HP)
8adf6ba8f5
Add merged label on branches page
2015-08-11 23:00:46 +02:00