You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ponymail.apache.org by sebbASF <gi...@git.apache.org> on 2016/11/03 15:46:24 UTC
[GitHub] incubator-ponymail issue #221: Bug: pminfo.lua is not self-consistent; rejec...
GitHub user sebbASF opened an issue:
https://github.com/apache/incubator-ponymail/issues/221
Bug: pminfo.lua is not self-consistent; rejects valid list names
pminfo.lua can return inconsistent data, for example:
```{"no_threads":18,"participants":5,"active_lists":{},"no_active_lists":0,"hits":28,...```
It does not make sense to have 28 messages and 18 threads if there are no active lists.
This is caused by the following list/domain name validation:
```if not domain:match("%..-%..-%..-") and domain:match("^[-_a-z0-9.]+$") and list:match("^[-_a-z0-9.]+$") then```
which is only applied when processing the active lists.
Is there really a need to validate the list names here?
If so, then the other statistics need adjusting accordingly.
Assuming the validation is needed, then it should be done using a common utility method (DRY) and should not reject list names such as
dev@ponymail.incubator.apache.org
This currently fails the check:
```not domain:match("%..-%..-%..-")'```
----
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ponymail issue #221: Bug: pminfo.lua is not self-consistent; rejec...
Posted by sebbASF <gi...@git.apache.org>.
Github user sebbASF commented on the issue:
https://github.com/apache/incubator-ponymail/issues/221
FIxed as part of #282
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-ponymail issue #221: Bug: pminfo.lua is not self-consistent; rejec...
Posted by sebbASF <gi...@git.apache.org>.
Github user sebbASF closed the issue at:
https://github.com/apache/incubator-ponymail/issues/221
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---