Skip to content

Commit

Permalink
Default the windows installer to an empty nodename (#347)
Browse files Browse the repository at this point in the history
It's a better default to use the system's hostname.

Fixes #339

(cherry picked from commit 5472b0d)
  • Loading branch information
mpfz0r committed Aug 13, 2019
1 parent 8efde7e commit 6b5d516
Showing 1 changed file with 6 additions and 8 deletions.
14 changes: 6 additions & 8 deletions dist/recipe.nsi
Original file line number Diff line number Diff line change
Expand Up @@ -292,14 +292,16 @@ Function nsDialogsPage
${NSD_CreateText} 50 20 75% 12u "http://127.0.0.1:9000/api"
Pop $InputServerUrl

${NSD_CreateLabel} 0 60 100% 12u "Enter the name of this instance:"
${NSD_CreateLabel} 0 60 100% 12u "The name of this instance:"
Pop $Label
${NSD_CreateText} 50 80 75% 12u "graylog-sidecar"
${NSD_CreateText} 50 80 75% 12u ""
Pop $InputNodeName
${NSD_CreateLabel} 50 100 100% 12u "If empty, the hostname will be used."
Pop $Label

${NSD_CreateLabel} 0 120 100% 12u "Enter the server API token:"
${NSD_CreateLabel} 0 140 100% 12u "Enter the server API token:"
Pop $Label
${NSD_CreateText} 50 140 75% 12u ""
${NSD_CreateText} 50 160 75% 12u ""
Pop $InputApiToken

nsDialogs::Show
Expand All @@ -314,10 +316,6 @@ Function nsDialogsPageLeave
MessageBox MB_OK "Please enter a valid address to your Graylog server!"
Abort
${EndIf}
${If} $NodeName == ""
MessageBox MB_OK "Please enter the instance name!"
Abort
${EndIf}
${If} $ApiToken == ""
MessageBox MB_OK "Please enter an API token!"
Abort
Expand Down

0 comments on commit 6b5d516

Please sign in to comment.