maps String.strip over the lines of the peerlist file
Explain how you tested your changes:
I haven't been able to get a locally compiled node to run, but I've confirmed that adding spaces to the peer list without this change does change the error message and with this change no longer changes the error message.
I've used this branch to run a node using a peerlist with trailing spaces
Checklist:
[x] Dependency versions are unchanged
Notify Velocity team if dependencies must change in CI
[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it
Mention expected invariants, implicit constraints
[ ] Tests were added for the new behavior
Document test purpose, significance of failures
Test names should reflect their purpose
[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
rbonichon
commented
2 weeks ago
Explain your changes:
String.stripover the lines of the peerlist fileExplain how you tested your changes:
I haven't been able to get a locally compiled node to run, but I've confirmed that adding spaces to the peer list without this change does change the error message and with this change no longer changes the error message.Checklist:
[x] Dependency versions are unchanged
[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it
[ ] Tests were added for the new behavior
[x] All tests pass (CI will check this if you didn't)
[x] Serialized types are in stable-versioned modules
[x] Does this close issues? List them
Closes #15696