openboxes / openboxes/openboxes

Bug Report feature reports email successfully sent even though email is disabled

Open
#189 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

status: needs investigation type: bug
Dominant language
Groovy
Stars
899
Forks
503
Avg merge
3d 7h
Merged PRs (30d)
54

Description

2017-03-31 16:19:30,959 [ajp-bio-8009-exec-5] INFO  core.MailService  - Sending email with attachment [errors@openboxes.com]
2017-03-31 16:19:30,959 [ajp-bio-8009-exec-5] INFO  core.MailService  - grailsApplication.config.grails.mail.enabled 'false'
2017-03-31 16:19:30,960 [ajp-bio-8009-exec-5] INFO  core.MailService  - Mail is disabled
2017-03-31 16:19:30,963 [ajp-bio-8009-exec-5] INFO  filters.SecurityFilters  - Request duration for (errors/processError): 36ms/0ms

bug report features reports success even though email is disabled

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with core.MailService and the errors/processError request shown in the log. Reproduce the report with grails.mail.enabled set to false, then trace why the UI reports success despite the “Mail is disabled” path. Done means the disabled-email case is reported accurately; no test or file is named in the issue.

Written by the indexing model from the issue text.

Assessment

Tech stack
groovy
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.