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

Tweak initialization INFO log of rollup-http-server #11

Open
tuler opened this issue May 19, 2023 · 0 comments
Open

Tweak initialization INFO log of rollup-http-server #11

tuler opened this issue May 19, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@tuler
Copy link
Member

tuler commented May 19, 2023

Today when the server is initiated this is the INFO log the is printed:

[INFO  rollup_http_server] starting http dispatcher service...
[INFO  rollup_http_server::http_service] starting http dispatcher http service!
[INFO  actix_server::builder] starting 1 workers
[INFO  actix_server::server] Actix runtime found; starting in Actix runtime
[INFO  rollup_http_server::dapp_process] starting dapp: node index.js

Lines 1 and 2 are kind of duplicated. Maybe remove line 2.
It would be nice to add an INFO saying the service was started, and on which hostname and port, like;

service listening at http://localhost:5004
@edubart edubart added the enhancement New feature or request label May 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Todo
Development

No branches or pull requests

2 participants