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

(1.11.x) max_contacts is ignored #492

Closed
bbuitenhuis opened this issue May 1, 2015 · 8 comments
Closed

(1.11.x) max_contacts is ignored #492

bbuitenhuis opened this issue May 1, 2015 · 8 comments
Assignees

Comments

@bbuitenhuis
Copy link
Contributor

I don't have any proof or logs to back this up. But with the latest version LTS: 1.11.4 , this variable looks ignored. With git revision: 80793b7 (version 1.11.2) this problem won't occur.

@bogdan-iancu bogdan-iancu self-assigned this May 4, 2015
@bogdan-iancu
Copy link
Member

@bbuitenhuis , I checked the history for both usrloc and registrar modules and I found no changes related to the max_contacts.
Now, are you referring to max_contacts the module parameter or to the save() function param ?
What db mode do you use for usrloc ?
And by "ignored" you mean whatever value you set, it does nothing ? like keep accepting whatever number of contacts per AOR ?

@bbuitenhuis
Copy link
Contributor Author

@bogdan-iancu , we are using the following parameters:
modparam("registrar", "max_contacts", 1)

In the test I'm using:
modparam("usrloc", "db_mode",3)

With "ignored" mean that save() is adding multiple contacts to the location database, and from my point of view is ignoring the max_contacts.

@bogdan-iancu
Copy link
Member

Do you use multiple OpenSIPS instances over the same location table ?

@bbuitenhuis
Copy link
Contributor Author

@bogdan-iancu No its only one OpenSips instance that is using this table/database.

@bogdan-iancu
Copy link
Member

Just tested and, as expected, I get:
May 7 14:23:22 [22609] INFO:registrar:update_contacts: too many contacts for AOR , max=1
Second registration was declined and nothing added to DB.

@bogdan-iancu
Copy link
Member

@bbuitenhuis , do you have any updates or info on this ?

@bbuitenhuis
Copy link
Contributor Author

@bogdan-iancu No sorry, I can replicate this anymore (with a fresh installation).
I think we can close this issue for now.

@bogdan-iancu
Copy link
Member

ok @bbuitenhuis , let's close it then...

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

No branches or pull requests

2 participants