Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

modules/ipops: fix subnet matching for mask /0 #704

Merged
merged 2 commits into from Jul 11, 2016

Conversation

furmur
Copy link
Contributor

@furmur furmur commented Jul 11, 2016

@miconda
Copy link
Member

miconda commented Jul 11, 2016

So the pull request has a commit with a fix and a commit with a new feature, right?

@furmur
Copy link
Contributor Author

furmur commented Jul 11, 2016

right

  • fix: matching for /0
  • feature: support for comma-separated list of networks

i thought that new commits have to be added to the pull-request manually,
but now i see that last commit was automatically added to the pull-request.
sorry if it caused a little mess.

On 07/11/2016 04:41 PM, Daniel-Constantin Mierla wrote:

So the pull request has a commit with a fix and a commit with a new
feature, right?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#704 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AKGuBWwNcWr8YwYh4qDzuA9xm_QDETQWks5qUkgegaJpZM4JJOqx.

Regards
Michael Furmur

@miconda
Copy link
Member

miconda commented Jul 11, 2016

OK. I'll merge, thanks!

@miconda miconda merged commit 7bfdbdf into kamailio:master Jul 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants