Stefan Sperling has posted comments on this change. ( 
https://gerrit.osmocom.org/12228 )

Change subject: disable recording of LU timestamps by default
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/#/c/12228/2/src/hlr_vty.c
File src/hlr_vty.c:

https://gerrit.osmocom.org/#/c/12228/2/src/hlr_vty.c@133
PS2, Line 133:  vty_out(vty, "Timestamps of Location Updates will be stored in 
the HLR database.%s", VTY_NEWLINE);
> IMHO these vty_out lines can be dropped, but fine otherwise. […]
Can you elaborate what you're referring to when you say we're storing 
subscriber names?
Which field of the DB do you mean? Do you mean some other database then the HLR 
db?



--
To view, visit https://gerrit.osmocom.org/12228
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-hlr
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2f18b5ddc9b4b8e1174c6dea71cddf1c8d2230df
Gerrit-Change-Number: 12228
Gerrit-PatchSet: 2
Gerrit-Owner: Stefan Sperling <[email protected]>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Stefan Sperling <[email protected]>
Gerrit-CC: Pau Espin Pedrol <[email protected]>
Gerrit-Comment-Date: Mon, 10 Dec 2018 17:06:20 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No

Reply via email to