summaryrefslogtreecommitdiff
path: root/lib/gitlab/email
Commit message (Collapse)AuthorAgeFilesLines
* Adds the Rubocop ReturnNil copAndrew Newdigate2019-03-061-1/+1
| | | | | This style change enforces `return if ...` instead of `return nil if ...` to save maintainers a few minor review points
* Prevent comments by email when issue is lockedHeinrich Lee Yu2019-01-311-1/+1
| | | | | | | | | This changes the permission check so it uses the policy on Noteable instead of Project. This prevents bypassing of rules defined in Noteable for locked discussions and confidential issues. Also rechecks permissions when reply_to_discussion_id is provided since the discussion_id may be from a different noteable.
* Remove caching of CSV file49231-import-issues-csvHeinrich Lee Yu2019-01-071-2/+2
| | | | Load whole file in memory to simplify code
* Refactor upload service to return uploaderHeinrich Lee Yu2019-01-071-1/+1
| | | | Also changes old calls to the service
* Fix already initialized constant constant warning29951-issue-creation-by-email-without-subaddressingBrett Walker2019-01-034-4/+4
|
* Move constant definitionBrett Walker2019-01-034-4/+4
|
* Refactoring and review commentsBrett Walker2019-01-034-30/+35
| | | | including verifying the project_slug
* Address review feedbackBrett Walker2019-01-032-4/+7
|
* Use new issue email address formatBrett Walker2019-01-034-10/+31
| | | | We now use `-issue` in order to support catch all email addresses
* Use new merge request email address formatBrett Walker2019-01-032-20/+34
| | | | | We now use `-merge-request` instead of `+merge-request+` in order to support catch all email addresses
* Use new unsubscribe linkBrett Walker2019-01-031-2/+10
| | | | | We now use `-unsubscribe` instead of `+unsubscribe` in order to support catch all email addresses
* Merge branch 'frozen-string-lib-gitlab-more' into 'master'Rémy Coutable2018-11-099-1/+21
|\ | | | | | | | | Enable even more frozen string in lib/gitlab See merge request gitlab-org/gitlab-ce!22863
| * Enable even more frozen string in lib/gitlabgfyoung2018-11-069-1/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enables frozens string for the following: * lib/gitlab/conflict/**/*.rb * lib/gitlab/cross_project_access/**/*.rb * lib/gitlab/cycle_analytics/**/*.rb * lib/gitlab/data_builder/**/*.rb * lib/gitlab/database/**/*.rb * lib/gitlab/dependency_linker/**/*.rb * lib/gitlab/diff/**/*.rb * lib/gitlab/downtime_check/**/*.rb * lib/gitlab/email/**/*.rb * lib/gitlab/etag_caching/**/*.rb Partially addresses gitlab-org/gitlab-ce#47424.
* | Apply patches when creating MR via emailBob Van Landuyt2018-11-072-2/+49
|/ | | | | | | | | | | | | | | This allows users to add patches as attachments to merge request created via email. When an email to create a merge request is sent, all the attachments ending in `.patch` will be applied to the branch specified in the subject of the email. If the branch did not exist, it will be created from the HEAD of the repository. When the patches could not be applied, the error message will be replied to the user. The patches can have a maximum combined size of 2MB for now.
* Disable existing offenses for the CodeReuse copsYorick Peterse2018-09-112-0/+4
| | | | | This whitelists all existing offenses for the various CodeReuse cops, of which most are triggered by the CodeReuse/ActiveRecord cop.
* Remove requires from email handlersSean McGivern2018-09-031-5/+0
|
* Add frozen_string_literal: true to email handlersSean McGivern2018-09-037-0/+14
|
* Extract EE-specific lines from Gitlab::Email::HandlerSean McGivern2018-09-031-7/+13
|
* Merge branch 'security-event-counters-private-data' into 'master'Felipe Artur Cardozo2018-07-244-16/+0
|\ | | | | | | | | [master] Don't expose project names in various counters See merge request gitlab/gitlabhq!2418
| * Don't expose project names in various countersYorick Peterse2018-06-214-16/+0
| | | | | | | | | | | | | | Various counters would expose either project names, or full project paths (e.g. "gitlab-org/gitlab-ce"). This commit changes various places where we use "add_event" so we no longer expose (potentially) private information.
* | Implement action mailer delivery metrics observerGrzegorz Bizon2018-07-161-0/+31
| |
* | Move existing email interceptors into the same moduleGrzegorz Bizon2018-07-163-0/+43
|/
* skip email trim when email is creating new issueCameron Crockett2018-05-223-5/+12
| | | | | | | | | | | | | | | Updates from MR discussion 1. Added test for ReplyParser 2. Changed param to trim_reply with default set as true Removed keyword param in favor of normal options param updates for MR discussion Resolutions for code review comments more code review fixes
* Backport 5480-epic-notifications from EEMario de la Ossa2018-05-072-3/+8
|
* Correct permissions for creating merge requests from issuesBob Van Landuyt2018-04-111-1/+2
| | | | | | | This could only be possible for users that can create merge requests within a project. So they need to be a allowed to create a branch and create a merge request.
* Make email handler clearerRiccardo Padovani2018-04-091-1/+1
|
* Enable RuboCop Style/RegexpLiteralTakuya Noguchi2018-02-011-1/+1
|
* Adds Rubocop rule for line break around conditionals🙈 jacopo beschi 🙉2018-01-111-0/+1
|
* Use message body for merge request descriptionjprovazn-descriptionJan Provaznik2017-12-111-1/+3
| | | | | | | | | | | | | When a merge request is created from email, use message body as merge request description. If message body is empty then merge request description is still created from the source branch commit (if there is only single commit in the merge request). If message body is empty and there are multiple commits in the source branch, then merge request description is left empty. Closes #40968
* Create merge request from emailJan Provaznik2017-12-053-2/+73
| | | | | | | | | | | * new merge request can be created by sending an email to the specific email address (similar to creating issues by email) * for the first iteration, source branch must be specified in the mail subject, other merge request parameters can not be set yet * user should enable "Receive notifications about your own activity" in user settings to receive a notification about created merge request Part of #32878
* Adds Rubocop rule for line break after guard clauseJacopo2017-11-161-0/+1
| | | | Adds a rubocop rule (with autocorrect) to ensure line break after guard clauses.
* Check against "Auto-Submitted: no" insteadLin Jen-Shin2017-08-031-2/+11
| | | | | | | | | | This would be much more accurate. We assume this is an auto-generated email if such header is provided, and the value is not "no". It could also be: "auto-generated", "auto-replied", or other values from extension. It seems that only "no" could mean that this is sent by a human. See: https://tools.ietf.org/html/rfc3834
* Don't send rejection mails for all auto-generated mails28472-ignore-auto-generated-mailsLin Jen-Shin2017-08-032-1/+4
| | | | Also make it easier to have mailer helper
* Rename many path_with_namespace -> full_pathGabriel Mazetto2017-08-011-1/+1
|
* Include new URL helpers retroactively into includers of Gitlab::Routingdm-fix-project-path-helpers-productionDouwe Maan2017-07-071-1/+1
|
* Create and use project path helpers that only need a project, no namespaceDouwe Maan2017-07-051-10/+3
|
* unwrap all links with no hrefhttp://jneen.net/2017-06-191-0/+7
|
* Consistent diff and blob size limit namesDouwe Maan2017-05-291-1/+1
|
* Fix email receiver metrics eventsfix-email-receiver-metricsSean McGivern2017-05-053-3/+3
| | | | `Project#inspect` isn't very useful for examining usage of these features.
* Merge branch '12910-uploader-pers-snippet' into 'master' Sean McGivern2017-05-041-1/+1
|\ | | | | | | | | Prepare uploaders for personal snippets comments See merge request !11022
| * Support uploaders for personal snippets comments12910-uploader-pers-snippetJarka Kadlecova2017-05-021-1/+1
| |
* | Handle incoming emails from aliases correctly2247-emails-forwarded-to-service-desk-email-don-t-comeSean McGivern2017-05-031-5/+11
| | | | | | | | | | These set the 'actual' destination email in one of the Delivered-To lines, so check those too.
* | Don't blow up when email has no References headerdont-blow-up-when-email-has-no-references-headerSean McGivern2017-05-011-0/+2
|/ | | | | | If an email doesn't match our incoming email patterns on the To header, we fall back to the References header. If there was no References header, we'd raise an exception, when we'd be better off acting as if it was empty.
* Make handler metric params more explicitemail-handler-metricsSean McGivern2017-04-215-5/+17
|
* Store projects in metrics for email repliesSean McGivern2017-04-202-4/+4
|
* Add metrics events for incoming emailsSean McGivern2017-04-201-0/+4
| | | | | This lets us track how many incoming emails a GitLab instance is processing, by email type (handler) and by project (where applicable).
* Merge branch 'master' into new-resolvable-discussionLuke "Jared" Bennett2017-04-071-1/+5
|\
| * Backport permissions and multi-line array to CEFelipe Artur2017-04-071-1/+5
| |
* | Add specsDouwe Maan2017-04-051-6/+1
| |
* | Add option to start a new discussion on an MRDouwe Maan2017-04-051-7/+1
|/
View Lorry Controller Status