You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by kgiusti <gi...@git.apache.org> on 2018/10/08 13:25:39 UTC
[GitHub] qpid-dispatch pull request #388: DISPATCH-1131: add better validation of add...
GitHub user kgiusti opened a pull request:
https://github.com/apache/qpid-dispatch/pull/388
DISPATCH-1131: add better validation of address patterns and prefixes
This moves the address pattern/prefix validation to a common function
that can be share by both link routes add configured addresses.
This closes #385
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kgiusti/dispatch DISPATCH-1131
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/qpid-dispatch/pull/388.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #388
----
commit c38dde95d5f32a743f41a8a8c728a8c554bf17ff
Author: Kenneth Giusti <kg...@...>
Date: 2018-09-27T17:22:51Z
DISPATCH-1131: add better validation of address patterns and prefixes
This moves the address pattern/prefix validation to a common function
that can be share by both link routes add configured addresses.
This closes #385
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] qpid-dispatch pull request #388: DISPATCH-1131: add better validation of add...
Posted by kgiusti <gi...@git.apache.org>.
Github user kgiusti closed the pull request at:
https://github.com/apache/qpid-dispatch/pull/388
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] qpid-dispatch issue #388: DISPATCH-1131: add better validation of address pa...
Posted by codecov-io <gi...@git.apache.org>.
Github user codecov-io commented on the issue:
https://github.com/apache/qpid-dispatch/pull/388
# [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=h1) Report
> Merging [#388](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=desc) into [master](https://codecov.io/gh/apache/qpid-dispatch/commit/3f92be9015615d119cd4707e0ff63a2cf1a73a7c?src=pr&el=desc) will **increase** coverage by `0.11%`.
> The diff coverage is `92.98%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/qpid-dispatch/pull/388/graphs/tree.svg?width=650&token=rk2Cgd27pP&height=150&src=pr)](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #388 +/- ##
==========================================
+ Coverage 84.93% 85.05% +0.11%
==========================================
Files 73 73
Lines 16499 16523 +24
==========================================
+ Hits 14014 14054 +40
+ Misses 2485 2469 -16
```
| [Impacted Files](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/router\_core/agent\_config\_link\_route.c](https://codecov.io/gh/apache/qpid-dispatch/pull/388/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2NvbmZpZ19saW5rX3JvdXRlLmM=) | `86.74% <100%> (+7.49%)` | :arrow_up: |
| [src/router\_core/route\_control.c](https://codecov.io/gh/apache/qpid-dispatch/pull/388/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX2NvbnRyb2wuYw==) | `96.03% <100%> (-0.14%)` | :arrow_down: |
| [src/router\_core/agent\_config\_address.c](https://codecov.io/gh/apache/qpid-dispatch/pull/388/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2NvbmZpZ19hZGRyZXNzLmM=) | `78.82% <91.48%> (+2.5%)` | :arrow_up: |
| [src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/388/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=) | `92.05% <0%> (-0.34%)` | :arrow_down: |
| [src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/388/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=) | `90.52% <0%> (-0.16%)` | :arrow_down: |
| [src/parse.c](https://codecov.io/gh/apache/qpid-dispatch/pull/388/diff?src=pr&el=tree#diff-c3JjL3BhcnNlLmM=) | `86.1% <0%> (+0.27%)` | :arrow_up: |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=footer). Last update [3f92be9...31bb414](https://codecov.io/gh/apache/qpid-dispatch/pull/388?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org