| OCram | |
| Jul 9 2015, 8:38 AM |
| F1137423: screenshot.png | |
| Mar 2 2016, 1:08 AM |
| F687053: Screen Shot 2015-08-03 at 1.41.59 PM.png | |
| Aug 3 2015, 9:04 PM |
| F641867: pasted_file | |
| Jul 20 2015, 6:58 AM |
| F630949: Screen Shot 2015-07-17 at 7.30.46 AM.png | |
| Jul 17 2015, 2:32 PM |
| F630947: Screen Shot 2015-07-17 at 7.30.10 AM.png | |
| Jul 17 2015, 2:32 PM |
| F603615: pasted_file | |
| Jul 10 2015, 5:47 AM |
| F603613: pasted_file | |
| Jul 10 2015, 5:47 AM |
| F600905: pasted_file | |
| Jul 9 2015, 8:38 AM |
Hello together,
I set up a Herald rule to prevent some users to do changes on a master branch.
The blocking mechanism itself works, but no mails are sent.
I already checked the maillog on the host. Mailing works in general. Other Herald rules can send mails also.
So I'm not shure if I missed anything.
I'm currently using phabricator in the version ab7dc13c67fde3c1fd53dfa945b940e010e5e8f9
Can you first verify this bug is not resolved at HEAD? https://secure.phabricator.com/book/phabcontrib/article/bug_reports/#update-phabricator
I can't reproduce this. Here's a rule I wrote:
After pushing a commit, here's the transcript:
Mail is generated and sent, and appears in the outbound mail log, e.g. bin/phd list-outbound:
$ ./bin/mail show-outbound --id 3743 PROPERTIES ID: 3743 Status: sent Related PHID: PHID-PSHE-m7sxhsxgly6cig6h4e4w Message: PARAMETERS sensitive: 1 subject-prefix: [Diffusion] vary-subject-prefix: [Push] subject: (1 commit, master) rTESTYb1ec79ed0ac5 spaghetti from: PHID-USER-cvfydnwadpdj7vdon36z thread-id: PHID-PSHE-m7sxhsxgly6cig6h4e4w is-first-message: 1 is-bulk: 1 to: ["PHID-USER-hwsda37inl2orub6qxkw"] HEADERS Thread-Topic: (1 commit, master) rTESTYb1ec79ed0ac5 spaghetti X-Phabricator-To: <PHID-USER-hwsda37inl2orub6qxkw> RECIPIENTS example-list (Example List) TEXT BODY admin pushed a commit to rTESTY. DETAILS http://local.phacility.com/diffusion/pushlog/view/80/ COMMITS rTESTYb1ec79ed0ac5 (master) spaghetti REFERENCES branch master 74780a9ef32e > b1ec79ed0ac5 To: example-list HTML BODY (This message has no HTML body.)
Note that in this case and in the T8863 case, mail recipients must now be able to see objects (e.g., revisions and repositories) to receive email about them. It's possible that that's the source of this issue.
For example, if neither "admin" or the list have permission to see the repository (or, in the case of T8863, if the list can't see the revision or the repository it belongs to), they won't get any mail. Is it possible that's the issue?
Ah, yeah, I believe that's the issue, I thought the mailing list user doesn't have to be given the read permission specifically. Thanks a lot for looking!
Tested it again with c3e16e26c9fd7acdfa5c6a12d8c591ac46d7a85
I checked if the members who should get the message already joined the repository and have the required permissions.
But I still seen no entry in bin/phd list-outbound.
And the transcript looks fine:
For the original issue: when a change is blocked, we do not send email about it. There is no way to be alerted to blocked pushes right now, and I generally don't think this is useful enough to include in the upstream. This interaction could possibly be more clear, but feels like something we just stick in the documentation to me.
For the visibility issue, after T8726 the transcript will show the issue more clearly:
There is no way to be alerted to blocked pushes right now, and I generally don't think this is useful enough to include in the upstream. This interaction could possibly be more clear, but feels like something we just stick in the documentation to me.
Is this still the case? If so, I think it would help if combinations of the "Block push" and "Send email" actions were impossible. It's not at all clear from the following transcript why no e-mail is actually being generated.
