Skip to content
This repository was archived by the owner on Dec 10, 2025. It is now read-only.

Conversation

@twisti-dev
Copy link
Contributor

This pull request includes several changes to update dependencies, refactor code, and improve the project structure. The most important changes include updating the Netty version, adding a new Netty module, and refactoring how dependencies are bundled and imported.

Dependency updates and additions:

  • Updated netty version from 4.1.117.Final to 4.2.0.Final and added netty-tcnative version 2.0.70.Final in gradle/libs.versions.toml.
  • Added netty-tcnative module reference in gradle/libs.versions.toml.

Dependency bundling:

  • Added netty-all bundle to include both netty-all and netty-tcnative in gradle/libs.versions.toml.
  • Updated surf-cloud-api/surf-cloud-api-common/build.gradle.kts to use the new netty-all bundle instead of individual netty dependencies.

Code refactoring:

  • Removed unused import logger from ServerEncryptionManager.kt. [1] [2]

@twisti-dev twisti-dev added this to the Pre-release milestone Apr 9, 2025
@twisti-dev twisti-dev requested a review from Copilot April 9, 2025 11:19
@twisti-dev twisti-dev self-assigned this Apr 9, 2025
@twisti-dev twisti-dev linked an issue Apr 9, 2025 that may be closed by this pull request
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • surf-cloud-api/surf-cloud-api-common/build.gradle.kts: Language not supported

@twisti-dev twisti-dev linked an issue Apr 9, 2025 that may be closed by this pull request
@twisti-dev twisti-dev merged commit 2b41676 into master Apr 9, 2025
@twisti-dev twisti-dev linked an issue Apr 9, 2025 that may be closed by this pull request
@twisti-dev twisti-dev deleted the 10-use-aes-instead-of-rsa branch April 9, 2025 11:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Diffie-Hellman Key Exchange in Runtime Use AES instead of RSA Simplify client certification creation

2 participants