Skip to content

Commit

Permalink
set Module-Failure-Message, not Reply-Message
Browse files Browse the repository at this point in the history
  • Loading branch information
alandekok committed Dec 31, 2015
1 parent e445fe7 commit e61108b
Showing 1 changed file with 12 additions and 12 deletions.
24 changes: 12 additions & 12 deletions raddb/policy.d/filter
Expand Up @@ -29,8 +29,8 @@ filter_username {
# e.g. "user@ site.com", or "us er", or " user", or "user "
#
if (&User-Name =~ / /) {
update reply {
&Reply-Message += 'Rejected: Username contains whitespace'
update request {
&Module-Failure-Message += 'Rejected: Username contains whitespace'
}
reject
}
Expand All @@ -40,8 +40,8 @@ filter_username {
# e.g. "user@site.com@site.com"
#
if (&User-Name =~ /@[^@]*@/ ) {
update reply {
&Reply-Message += 'Rejected: Multiple @ in username'
update request {
&Module-Failure-Message += 'Rejected: Multiple @ in username'
}
reject
}
Expand All @@ -51,8 +51,8 @@ filter_username {
# e.g. "user@site..com"
#
if (&User-Name =~ /\.\./ ) {
update reply {
&Reply-Message += 'Rejected: Username contains ..s'
update request {
&Module-Failure-Message += 'Rejected: Username contains ..s'
}
reject
}
Expand All @@ -62,8 +62,8 @@ filter_username {
# e.g. "user@site.com"
#
if ((&User-Name =~ /@/) && (&User-Name !~ /@(.+)\.(.+)$/)) {
update reply {
&Reply-Message += 'Rejected: Realm does not have at least one dot separator'
update request {
&Module-Failure-Message += 'Rejected: Realm does not have at least one dot separator'
}
reject
}
Expand All @@ -73,8 +73,8 @@ filter_username {
# e.g. "user@site.com."
#
if (&User-Name =~ /\.$/) {
update reply {
&Reply-Message += 'Rejected: Realm ends with a dot'
update request {
&Module-Failure-Message += 'Rejected: Realm ends with a dot'
}
reject
}
Expand All @@ -84,8 +84,8 @@ filter_username {
# e.g. "user@.site.com"
#
if (&User-Name =~ /@\./) {
update reply {
&Reply-Message += 'Rejected: Realm begins with a dot'
update request {
&Module-Failure-Message += 'Rejected: Realm begins with a dot'
}
reject
}
Expand Down

0 comments on commit e61108b

Please sign in to comment.