| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
The 'assigned' reason doesn't apply to notes, but the other two
can ('mentioned' and 'own_activity'), so we can still use this for note
emails.
|
|\ |
|
| |\
| | |
| | |
| | |
| | | |
Prevent disclosure of merge request id via email
See merge request gitlab/gitlabhq!3313
|
| | |
| | |
| | |
| | |
| | | |
Do not disclosure merge request id via email for unauthorized users
when closing issues.
|
|/ /
| |
| |
| |
| | |
It should not be a start to a new thread but rather
a reply to an existing thread
|
| |
| |
| |
| |
| |
| | |
We encountered issues with setting module headers for namespaced models.
These changes address this. We retain the namespacing, but transform the
classnames to make them into safe email headers.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
**Prevention of running 2 simultaneous updates**
Instead of using `RemoteMirror#update_status` and raise an error if
it's already running to prevent the same mirror being updated at the
same time we now use `Gitlab::ExclusiveLease` for that.
When we fail to obtain a lease in 3 tries, 30 seconds apart, we bail
and reschedule. We'll reschedule faster for the protected branches.
If the mirror already ran since it was scheduled, the job will be
skipped.
**Error handling: Remote side**
When an update fails because of a `Gitlab::Git::CommandError`, we
won't track this error in sentry, this could be on the remote side:
for example when branches have diverged.
In this case, we'll try 3 times scheduled 1 or 5 minutes apart.
In between, the mirror is marked as "to_retry", the error would be
visible to the user when they visit the settings page.
After 3 tries we'll mark the mirror as failed and notify the user.
We won't track this error in sentry, as it's not likely we can help
it.
The next event that would trigger a new refresh.
**Error handling: our side**
If an unexpected error occurs, we mark the mirror as failed, but we'd
still retry the job based on the regular sidekiq retries with
backoff. Same as we used to
The error would be reported in sentry, since its likely we need to do
something about it.
|
|
|
|
|
| |
When sending access granted/rejected emails we should also respect
custom emails set for groups/sub-groups
|
|
|
|
|
| |
When a user's notification email is set for a group, we
should use that for pipeline emails
|
|
|
|
|
| |
This reverts to previous behavior where we have the group only when
we don't have a project
|
|
|
|
|
| |
A Noteable doesn't have a group directly, unless it's an epic - we need
to look for the project's group to find the right email address.
|
|
|
|
|
|
|
| |
Blocked admins should not receive notifications about failed repository
checks.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/63019
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
- Select notification email by walking up group/subgroup path
- Add settings UI to set group email notification address
- Add tests
|
|
|
|
| |
issue
|
|
|
|
| |
Part of single codebase changes.
|
|
|
|
|
| |
Backports https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/10161
(code out of ee/ folder).
|
|\
| |
| |
| |
| |
| |
| | |
Merge dev.gitlab.org master into GitLab.com master
Closes #2794, #2814, #2806, #2805, #2798, #2795, #2788, and #2787
See merge request gitlab-org/gitlab-ce!25746
|
| |\ |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Don't show new issue link after move
when a user does not have permissions
to display the new issue
|
|/ /
| |
| |
| |
| |
| |
| |
| | |
fix it ok
Fix
head pipeline
|
|/
|
|
|
|
|
|
|
|
|
| |
* Add email_header_and_footer_enabled flag to appearances table
* Set email_header_and_footer_enabled default value to false
* Add checkbox to appearance to toggle show header and footer in emails
* Add email_header_and_footer_enabled to allowed params in controller
* Add header and footer messages to the html and text email layouts
* Remove the color styling for emails header and footer
* Add empty_mailer layout for emails without layout,
to have the header and footer applied
|
|
|
|
| |
Process CSV uploads async using a worker then email results
|
|
|
|
| |
Also refactored issues and MRs to use the same email template file
|
| |
|
|\
| |
| |
| |
| |
| |
| |
| |
| | |
'54650-send-an-email-to-project-owners-when-a-mirror-update-fails' into 'master'
Send a notification email on mirror update errors
Closes #54650
See merge request gitlab-org/gitlab-ce!23595
|
| |
| |
| |
| |
| |
| | |
The email is sent to project maintainers containing the last mirror
update error. This will allow maintainers to set alarms and react
accordingly.
|
|/
|
|
| |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
|
|
|
| |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|