Fix a potential frozen string error in app/mailers/notify.rb
requested to merge 55183-frozenerror-can-t-modify-frozen-string-in-app-mailers-notify-rb into master
EE port: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/8791
What does this MR do?
Normally, we won't hit frozen string error since callers of Notify#mail_answer_note_thread
alls call Notify#subject
to generate the subject, which returns a non-frozen string.
That said, better be safe we had a case where Service Desk emails weren't using Notify#subject
to generate the subject and thus hitting the frozen string error when calling Notify#mail_answer_thread
(https://gitlab.com/gitlab-org/gitlab-ce/issues/55104 => https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/23683).
The EE port also adds test for the Service Desk emails: https://gitlab.com/gitlab-org/gitlab-ee/merge_requests/8791
What are the relevant issue numbers?
Closes #55183 (closed).
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary - [-] Documentation created/updated
-
Tests added for this feature/bug -
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides -
Conforms to the database guides - [-] Link to e2e tests MR added if this MR has Requires e2e tests label. See the Test Planning Process.
-
Security reports checked/validated by reviewer
Edited by Rémy Coutable