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

[HSS] Add hss_event_init after change to event system #2838

Merged
merged 2 commits into from
Jan 3, 2024

Conversation

jmasterfunk84
Copy link
Contributor

When use_mongodb_change_stream is enabled, the HSS is unable to start due to the following error:

[app] FATAL: hss_event_new: Assertion `e' failed. (../src/hss/hss-event.c:46)

In #2739, the event functions were moved into the hss app, however, hss_event_init is never called. This adds the call to the function to allow the hss to start with change streams enabled.

@jmasterfunk84 jmasterfunk84 changed the title Add hss_event_init after change to event system [HSS] Add hss_event_init after change to event system Dec 29, 2023
@acetcom
Copy link
Member

acetcom commented Dec 31, 2023

@jmasterfunk84

Please ensure that the 'hss_event_final()' function is invoked within the 'hss_terminate()' function in the 'hss-init.c' file.

Thanks a lot!
Sukchan

@jmasterfunk84
Copy link
Contributor Author

@acetcom

Thanks, I've made that change, too!

@acetcom acetcom merged commit ad2154a into open5gs:main Jan 3, 2024
5 checks passed
@jmasterfunk84 jmasterfunk84 deleted the MissingHSSEventInit branch January 3, 2024 16:19
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