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

server.Initialize receive an unnecessary argument #22

Closed
joselsegura opened this issue Jun 4, 2020 · 0 comments · Fixed by #23
Closed

server.Initialize receive an unnecessary argument #22

joselsegura opened this issue Jun 4, 2020 · 0 comments · Fixed by #23
Assignees

Comments

@joselsegura
Copy link
Collaborator

(copied from RedHatInsights/insights-results-aggregator#847)

The method server.Initialize has one string argument containing an address. This argument is only used to print a log message.

This log message is more or less the same as the one printed in server.Start, and the address that currently is taken from the argument can be safely extracted from server.Configuration.Address instead.

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 a pull request may close this issue.

1 participant