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
Fix/update json config #659
Conversation
jsonConfigs.getSection() returned a value when it needed to return a pointer to the jsonSection fields inside the struct. Even though the jsonSection type is a map, therefore on the heap, returning it as a value (non-pointer) resulted in it being disassociated with the jsonConfigs overarching struct. License: MIT Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
License: MIT Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
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.
shitty bug :(
License: MIT Signed-off-by: Adrian Lanzafame <adrianlanzafame92@gmail.com>
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.
small comment, way better like this
I don't know why the docker compose thing is failing but I'll check |
ah @lanzafame , docker compose is probably failing because the container |
License: MIT Signed-off-by: Hector Sanjuan <hector@protocol.ai>
Resolves #658.
jsonConfigs.getSection() returned a value when it needed to
return a pointer to the jsonSection fields inside the struct.
Even though the jsonSection type is a map, therefore on the heap,
returning it as a value (non-pointer) resulted in it being
disassociated with the jsonConfigs overarching struct.
License: MIT
Signed-off-by: Adrian Lanzafame adrianlanzafame92@gmail.com