Address review comments

Author: quentinmit

ansible/roles/real-postfix/files/postfix/force_pool

@@ -1,6 +1,10 @@
 # To force a vhost or user to one pool or the other, uncomment one of
 # the below:
+# Vhost to another pool:
 #/(.*)@vhost\.mit\.edu/ $1!vhost.mit.edu@[18.4.86.22]
+# User to another pool:
 #/user(\+.*)?@scripts\.mit\.edu/ user$1!scripts.mit.edu@[18.4.86.22]
+# Vhost to this pool:
 #/(.*)@vhost\.mit\.edu/ user+$1@localhost
+# User to this pool:
 #/user(\+.*)?@scripts\.mit\.edu/ user$1@localhost

ansible/roles/real-postfix/files/postfix/generic-strip-pool

@@ -1,5 +1,5 @@
-# Rewrite foo!bar.com@[18.4.86.200] to foo@bar.com
+# Rewrite foo!bar.com@[18.4.86.x] to foo@bar.com
 # This happens after transport selection, so the message is still
-# delivered to 18.4.86.200
+# delivered to 18.4.86.x
 
-/(.*)!(.*)@\[[0-9.]+\]/ $1@$2
+/^(.*)!(.*)@\[[0-9.]+\]$/ $1@$2

ansible/roles/real-postfix/files/postfix/pass-scripts.mit.edu

@@ -1,3 +1,3 @@
 # Match @scripts.mit.edu addresses (but don't change them)
 # This is chained with an LDAP lookup using pipemap
-/(.*)@scripts\.mit\.edu/ $1@scripts.mit.edu
+/^(.*)@scripts\.mit\.edu$/ $1@scripts.mit.edu

ansible/roles/real-postfix/files/postfix/pass-scripts.mit.edu-suffix

@@ -1,3 +1,3 @@
 # Match quentin+foo@scripts.mit.edu and rewrite to quentin@+foo
 # This allows the chained LDAP lookup to resolve quentin.scripts.mit.edu with %u
-/([^+]+)([^@]*)@scripts\.mit\.edu/ $1@$2
+/^([^+]+)([^@]*)@scripts\.mit\.edu$/ $1@$2