Commit 45788c4
committed
Don't crash on document URIs with an invalid scheme
A client can send a textDocument message whose URI uses a scheme Ruby's URI() parser rejects, such as the `_claude_fs_right:` scheme the Claude Code extension registers for inline diffs.
BaseServer#start only rescued Store::NonExistingDocumentError, so URI::InvalidURIError escaped the reader loop and crashed the server.
Rescue URI::Error there as well so the message falls through to normal dispatch.
Closes #40671 parent 4a6d31a commit 45788c4
2 files changed
Lines changed: 32 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
74 | 74 | | |
75 | 75 | | |
76 | 76 | | |
| 77 | + | |
| 78 | + | |
77 | 79 | | |
78 | 80 | | |
79 | 81 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1749 | 1749 | | |
1750 | 1750 | | |
1751 | 1751 | | |
| 1752 | + | |
| 1753 | + | |
| 1754 | + | |
| 1755 | + | |
| 1756 | + | |
| 1757 | + | |
| 1758 | + | |
| 1759 | + | |
| 1760 | + | |
| 1761 | + | |
| 1762 | + | |
| 1763 | + | |
| 1764 | + | |
| 1765 | + | |
| 1766 | + | |
| 1767 | + | |
| 1768 | + | |
| 1769 | + | |
| 1770 | + | |
| 1771 | + | |
| 1772 | + | |
| 1773 | + | |
| 1774 | + | |
| 1775 | + | |
| 1776 | + | |
| 1777 | + | |
| 1778 | + | |
| 1779 | + | |
| 1780 | + | |
| 1781 | + | |
1752 | 1782 | | |
1753 | 1783 | | |
1754 | 1784 | | |
| |||
0 commit comments