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
DeviceGroup, Subscribers Config Update changes #36
Conversation
proto/server/clientEvtHandler.go
Outdated
deleteConfigHss(client, val) | ||
} | ||
} | ||
|
||
for _, imsi := range devGroup.Imsis { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We end up in adding all IMSI's again...
e.g.
if device group had IMSI1, IMSI2
Now if I remove IMSI2 from the group, then with new code we send delete IMSI2. and then we are sending IMSI1 addition again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, you are right. i add logic to get newly added imsi list also.
… into sdcore-458-dev
e66ff25
to
7705a00
Compare
@@ -19,7 +19,6 @@ import ( | |||
|
|||
"github.com/free5gc/MongoDBLibrary" | |||
mongoDBLibLogger "github.com/free5gc/MongoDBLibrary/logger" | |||
//"github.com/free5gc/http2_util" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this
@@ -760,17 +760,24 @@ func DeleteSubscriberByID(c *gin.Context) { | |||
|
|||
ueId := c.Param("ueId") | |||
|
|||
filterUeIdOnly := bson.M{"ueId": ueId} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought this is already committed.
No description provided.