Skip to content

Commit

Permalink
Add -webSocket option by default when creating an inbound agent (#9665)
Browse files Browse the repository at this point in the history
* Add -webSocket option by default when creating an inbound agent

* Update core/src/main/resources/hudson/slaves/JNLPLauncher/main.properties

Co-authored-by: Jesse Glick <[email protected]>

---------

Co-authored-by: Jesse Glick <[email protected]>
  • Loading branch information
Vlatombe and jglick authored Aug 28, 2024
1 parent f84ab6a commit 5df30de
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
4 changes: 1 addition & 3 deletions core/src/main/java/hudson/slaves/JNLPLauncher.java
Original file line number Diff line number Diff line change
Expand Up @@ -213,9 +213,7 @@ private String getRemotingOptions(String computerName) {
sb.append("-name ");
sb.append(computerName);
sb.append(' ');
if (isWebSocket()) {
sb.append("-webSocket ");
}
sb.append("-webSocket ");
if (tunnel != null) {
sb.append(" -tunnel ");
sb.append(tunnel);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ slaveAgent.cli.run=Run from agent command line:
slaveAgent.cli.run.secret=Or run from agent command line, with the secret stored in a file:
powerShell.cli.curl=Note: PowerShell users must use curl.exe instead of curl because curl is a default PowerShell cmdlet alias for Invoke-WebRequest.
commonOptions=\
The most commonly used option is <code>-webSocket</code>. \
If you prefer to use TCP instead of WebSockets, remove the <code>-webSocket</code> option. \
Run <code>java -jar agent.jar -help</code> for more.
tcp-port-disabled=\
The TCP port is disabled so TCP agents may not be connected. \
Expand Down

0 comments on commit 5df30de

Please sign in to comment.