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

new ConnectedServerType in config | little improve #1760

Conversation

mrpalide
Copy link
Contributor

Did you run make format && make check? Yes

Fixes #_

Changes:

  • add alternative value to GetIP for check public IP of visor. (https://ip.plaintext.ir)
  • add new field to config file servers_type that determine what type of dmsg servers visor should connect. all, official and community are available values. Default value is all.

How to test this PR:

  • build
  • generate config, change servers_type value under dmsg config part to community
  • run visor, check connected dmsg server (available in logs)
  • stop visor, change servers_type to official, run again and check logs.
  • stop visor, change servers_type to all or empty or any other things (have same behavior), run again and check logs.

@0pcom 0pcom merged commit 9d80774 into skycoin:develop Mar 11, 2024
3 checks passed
@mrpalide mrpalide deleted the fix/handle-some-issues-on-skywire-before-release branch March 26, 2024 13:50
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