Skip to content

Commit

Permalink
Change default LS transport from 'stdio' to 'pipe'.
Browse files Browse the repository at this point in the history
Signed-off-by: Roland Grunberg <[email protected]>
  • Loading branch information
rgrunber committed Apr 24, 2024
1 parent 907a5ba commit 81d814b
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions src/javaServerStarter.ts
Original file line number Diff line number Diff line change
Expand Up @@ -58,10 +58,7 @@ export function prepareExecutable(requirements: RequirementsData, workspacePath,
executable.transport = TransportKind.stdio;
break;
default:
const isInsider: boolean = version.includes("insider");
const javaExtVersion = context.extension.packageJSON?.version;
const isPreReleaseVersion = /^\d+\.\d+\.\d{10}/.test(javaExtVersion);
executable.transport = (isInsider || isPreReleaseVersion) ? TransportKind.pipe : TransportKind.stdio;
executable.transport = TransportKind.pipe;
break;
}
logger.info(`Starting Java server with: ${executable.command} ${executable.args.join(' ')}`);
Expand Down

0 comments on commit 81d814b

Please sign in to comment.