-
Notifications
You must be signed in to change notification settings - Fork 16
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
feat: Store authentication keys in a separate database #63
feat: Store authentication keys in a separate database #63
Conversation
4ea0bf0
to
2da8610
Compare
@gatici, I am getting errors when running E2E tests using
|
Hello Gabriel, I am also checking and testing. |
c6976a6
to
0f1e50c
Compare
Hello Gabriel, We need to test this PR together with omec-project/webconsole#132. Otherwise both PR's will fail. Webconsole write the subsriber data into a second database by default Besides, I found a mistake in ConnectMongo function and fixed it. After testing 2 PRs together, simulation is successful.
|
UDR logs:
|
webconsole logs:
|
GNBSIM logs:
|
Hi Gulsum, Please take a look at the comment I left in the webconsole #132 |
producer/db_adapter.go
Outdated
// Connect to MongoDB | ||
if authurl == "" { |
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 see you are setting default here if not available. ..better place would be to set default in config parsing code.
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.
Done
b43fffa
to
0276eea
Compare
Signed-off-by: gatici <gulsum.atici@canonical.com>
0276eea
to
9a70585
Compare
Storing authentication keys in a separate database by using separate MongoDB clients.