Skip to content

feat(language-server): integrate LS#5878

Merged
Team-Hammerhead merged 1 commit intomainfrom
feat/automatic-upgrade-of-ls
May 7, 2025
Merged

feat(language-server): integrate LS#5878
Team-Hammerhead merged 1 commit intomainfrom
feat/automatic-upgrade-of-ls

Conversation

@Team-Hammerhead
Copy link
Contributor

@Team-Hammerhead Team-Hammerhead commented May 7, 2025

Changes since last integration of Language Server

commit be0352e868594be116a6ed1ac8004084a1331e4f
Author: Abdelrahman Shawki Hassan <shawki.hassan@snyk.io>
Date:   Tue May 6 15:33:58 2025 +0200

    fix: use request.Host to retrieve host header (#860)
    
    fix: use r.Host to retrieve host header

:100644 100644 39b52691 039dd1f3 M	internal/mcp/llm_binding.go
:100644 100644 bb8e9460 2ffe5edf M	internal/mcp/llm_binding_test.go

commit 9e64d5a60b5b708d4b3d372722e0e9cd60b88e6c
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Tue May 6 14:19:17 2025 +0200

    docs: synchronizing MCP README from snyk/user-docs (#859)
    
    Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

:100644 100644 d66e4aaf 0c67559a M	mcp_extension/README.md

commit 51ded6af0d4b67d4d585ec65f975658b0826cbc7
Author: Bastian Doetsch <bastian.doetsch@snyk.io>
Date:   Tue May 6 11:49:48 2025 +0200

    fix: copy auth link command should not use read locks (#858)
    
    * fix: copy auth link command should not use read locks
    
    The auth URL is only relevant during authentication. When authentication is triggered, it obtains a write-lock on the authentication service. Thus, the copy auth link command was waiting for authentication to finish until it obtained the now obsolete auth URL.
    
    This fix
    
    - adds a method to the Authentication Service interface and its implementations for retrieving the Auth URL via the provider
    - implements the retrieval without usage of any locks
    
    * fix: write test for AuthURL()
    
    * chore: add explanation to AuthURL why no lock is required
    
    * fix: ensure test cleanup

:100644 100644 21d8e48c 77a2ee96 M	.gitignore
:100644 100644 98483d94 3310cb5b M	domain/ide/command/copy_auth_link.go
:100644 100644 43a18806 9d237d69 M	infrastructure/authentication/auth_service.go
:100644 100644 a981896a 92ea38c4 M	infrastructure/authentication/auth_service_impl.go
:100644 100644 2e3250ff 8bbefe74 M	infrastructure/authentication/auth_service_impl_test.go
:100644 100644 90210095 be5851fb M	infrastructure/authentication/oauth_provider.go

commit 1e41ffb4695b8aab1a9cf6a8acdc2efd9bb79ce7
Author: Abdelrahman Shawki Hassan <shawki.hassan@snyk.io>
Date:   Tue May 6 10:51:33 2025 +0200

    fix: mcp host check [IDE-1119] (#856)
    
    * fix: add host header check in MCP middleware
    
    * fix: added tests
    
    * fix: parse host without port

:100644 100644 7f9c979f 39b52691 M	internal/mcp/llm_binding.go
:100644 100644 2d4e415d bb8e9460 M	internal/mcp/llm_binding_test.go

@Team-Hammerhead Team-Hammerhead requested a review from a team as a code owner May 7, 2025 08:51
@Team-Hammerhead Team-Hammerhead enabled auto-merge May 7, 2025 08:51
@github-actions
Copy link
Contributor

github-actions bot commented May 7, 2025

Warnings
⚠️

"feat: automatic integration of language server be0352e868594be116a6ed1ac8004084a1331e4f" is too long. Keep the first line of your commit message under 72 characters.

Generated by 🚫 dangerJS against 420d62a

@Team-Hammerhead Team-Hammerhead merged commit cf94a9f into main May 7, 2025
8 checks passed
@Team-Hammerhead Team-Hammerhead deleted the feat/automatic-upgrade-of-ls branch May 7, 2025 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants