Clients like neovim show an error message if `end` is received for a
token without having received a corresponding `begin`.

The specification also requires it:

> To start progress reporting a $/progress notification with the following payload must be sent

This ensures `begin` is always sent first and also avoids creating
intermediate `ProgressReport` instances.

rmuir added a commit to rmuir/nvim-lspconfig that referenced this pull request

rmuir mentioned this pull request

justinmk pushed a commit to neovim/nvim-lspconfig that referenced this pull request

Problem:
The nonstandard progress handler was added in #2153 but it can cause
some undesirable side-effects out of box ("press enter").
The original bug (mfussenegger/nvim-jdtls#327) was fixed upstream in the
language server three years ago (eclipse-jdtls/eclipse.jdt.ls#2258).

Solution:
Remove the workaround. Standard progress works.