- 11 5月, 2016 6 次提交
-
-
由 Sean McGivern 提交于
The repository push email can be very expensive to generate, especially with syntax-highlighted diffs. Instead of generating the email for each recipient, generate one email object and reset the Message-Id and To headers for each recipient. (Cloning would also be expensive in the case of large emails, although probably not as bad as generating from scratch.)
-
由 Sean McGivern 提交于
-
由 Stan Hu 提交于
-
由 Connor Shea 提交于
Also fixes the errors caused by enabling the cop. casecmp is more performant than `.downcase` and `==`.
-
由 Sean McGivern 提交于
`User#starred_projects` doesn't perform any visibility checks. This has a couple of problems: 1. It assumes a user can always view all of their starred projects in perpetuity (project not changed to private, access revoked, etc.). 2. It assumes that we'll only ever allow a user to star a project they can view. This is currently the case, but bugs happen. Add `User#viewable_starred_projects` to filter the starred projects by those the user either has explicit access to, or are public or internal. Then use that in all places where we list the user's starred projects.
-
由 Rémy Coutable 提交于
Signed-off-by: NRémy Coutable <remy@rymai.me>
-
- 10 5月, 2016 11 次提交
-
-
由 Dmitriy Zaporozhets 提交于
Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
由 Dmitriy Zaporozhets 提交于
Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
由 Grzegorz Bizon 提交于
-
由 Rémy Coutable 提交于
- Fix naming of API::CommitStatuses - Ensure we use require_dependency instead of require - Ensure the namespace is right in lib/api/api.rb, otherwise, we might require Grape::API::Helpers which defines the `#params` method. This is to avoid requiring a file multiple times and getting an "Already initialized constant" error. Signed-off-by: NRémy Coutable <remy@rymai.me>
-
由 Rémy Coutable 提交于
Signed-off-by: NRémy Coutable <remy@rymai.me>
-
由 Calin Seciu 提交于
-
由 Jacob Vosmaer 提交于
Robert Speicher and I believe this string gets sanitized further down the stack anyway. Doing this in a model class feels wrong.
-
由 Rémy Coutable 提交于
This reverts commit 4cc85a58, reversing changes made to 90ae445b. Signed-off-by: NRémy Coutable <remy@rymai.me>
-
由 Annabel Dunstone 提交于
-
由 Annabel Dunstone 提交于
-
由 Jeroen van Baarsen 提交于
In 8278b763 the default behaviour of annotation has changes, which was causing a lot of noise in diffs. We decided in #17382 that it is better to get rid of the whole annotate gem, and instead let people look at schema.rb for the columns in a table. Fixes: #17382
-
- 09 5月, 2016 8 次提交
-
-
由 Felipe Artur 提交于
-
由 Rémy Coutable 提交于
Signed-off-by: NRémy Coutable <remy@rymai.me>
-
由 cnam-dep 提交于
-
由 Grzegorz Bizon 提交于
-
由 Artem Sidorenko 提交于
-
由 Stan Hu 提交于
If you attempt to push thousands of branches at once, the 60-second timeout will occur because GitAccess checking does a lot of work to check if the user has permission to push to a branch. This changes does two things: 1. Instead of making 1 DB query per branch push, use a memoized list of protected branches to check 2. Memoize what permissions the user has to perform on this project On a test of 10,000 branch pushes, this prevents gitlab-shell from hitting the 60-second timeout. Closes #17225
-
由 Robert Speicher 提交于
-
由 Benedikt Huss 提交于
-
- 07 5月, 2016 5 次提交
-
-
由 Stan Hu 提交于
Closes #17348
-
由 Stan Hu 提交于
Run ProjectDestroyWorker after pending_delete attribute has been committed to DB
-
由 Stan Hu 提交于
Use a custom Loofah scrubber since sanitize 2.x transformers are inadequate to handle case-sensitive SVG attributes. sanitize parses documents as HTML instead of XML, which causes all SVG attribute names (e.g. viewBox) to be downcased. * SVG element list: https://www.w3.org/TR/SVG/eltindex.html * SVG attribute list: https://www.w3.org/TR/SVG/attindex.html Closes #14555
-
由 Ahmad Sherif 提交于
Fixes #14002
-
由 Dmitriy Zaporozhets 提交于
Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 06 5月, 2016 10 次提交
-
-
由 Jacob Schatz 提交于
-
由 Dmitriy Zaporozhets 提交于
Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
由 Annabel Dunstone 提交于
-
由 Dmitriy Zaporozhets 提交于
Signed-off-by: NDmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
由 Sean McGivern 提交于
-
由 Sean McGivern 提交于
If a branch starts with an issue's IID, followed by a hyphen, the description will be updated to say that is closes the issue. This also updates the title of the merge request to 'Resolves "$issue-title"', as long as: - There is more than one commit in the merge request (if there is only one commit, the commit's title will be used as before) - The issue's IID is valid for the project
-
由 Takuya Noguchi 提交于
-
由 Zeger-Jan van de Weg 提交于
-
由 Felipe Artur 提交于
-
由 Annabel Dunstone 提交于
-