mailhog / MailHog

Web and API based SMTP testing
MIT License
14.16k stars 1.08k forks source link

Web UI From {{ getSender(message) }} #224

Open Cyb10101 opened 6 years ago

Cyb10101 commented 6 years ago

If you send a E-Mail without "From", the error is still there in the frontend. I know you should not do it, but you can.

I would then simply put with a placeholder "From not set" or something like that. Also think of subject and to.

Reproduce bug run:

mhsendmail test@mailhog.local <<EOF
To: Test <test@mailhog.local>
Subject: Test message

Some content!
EOF

Open Webpage: http://127.0.0.1:8025 From is "{{ getSender(message) }}" .

Suitable for the issue: https://github.com/mailhog/MailHog/issues/147

kazik666 commented 3 years ago

The same situation is when From have first letter in lowercase.

JoshuaDoshua commented 3 years ago

JS Error

TypeError: Cannot read property '0' of undefined
    at ChildScope.$scope.getSender (controllers.js:194)
    at $parseFunctionCall (angular-1.3.8.js:12332)
    at Object.expressionInputWatch (angular-1.3.8.js:12735)
    at Scope.$digest (angular-1.3.8.js:14217)
    at Scope.$apply (angular-1.3.8.js:14488)
    at done (angular-1.3.8.js:9646)
    at completeRequest (angular-1.3.8.js:9836)
    at XMLHttpRequest.requestLoaded (angular-1.3.8.js:9777)
henri9813 commented 1 month ago

Hello,

The bug is still present in 2024.

image image