Skip to content

Use test containers for SFTP/Mina-sftp tests and remove certs from the repo#8618

Closed
gansheer wants to merge 6 commits into
apache:mainfrom
gansheer:feat/8534-openssh-certificat-auth-backport
Closed

Use test containers for SFTP/Mina-sftp tests and remove certs from the repo#8618
gansheer wants to merge 6 commits into
apache:mainfrom
gansheer:feat/8534-openssh-certificat-auth-backport

Conversation

@gansheer

@gansheer gansheer commented May 5, 2026

Copy link
Copy Markdown
Contributor

Ref #8534

Follow-up from #8629

  • Use dynamically generated certs instead of repo files
  • Use openssh-server container image instead of sshd-server (I found some limitations on certificates verification)

Notes:

  • still requires some refactoring

@gansheer gansheer force-pushed the feat/8534-openssh-certificat-auth-backport branch 2 times, most recently from fdf6636 to 4f3e29e Compare May 7, 2026 09:42
@gansheer gansheer marked this pull request as ready for review May 7, 2026 12:24
@gansheer gansheer force-pushed the feat/8534-openssh-certificat-auth-backport branch from 4f3e29e to fc53c6e Compare May 7, 2026 12:38
@gansheer gansheer requested a review from jamesnetherton May 7, 2026 12:38
@gansheer gansheer marked this pull request as draft May 7, 2026 12:54
@gansheer gansheer changed the title OpenSSH certificate-based authentication and host verification Use test containers for SFTP/Mina-sftp tests and remove certs from the repo May 7, 2026
@BuildStep
void registerForReflection(BuildProducer<ReflectiveClassBuildItem> reflectiveClasses) {
// Register standard Java security and SSHD classes
java.util.List<Class<?>> classes = new java.util.ArrayList<>(java.util.Arrays.asList(

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Add an import for java.util.List.

@gansheer

Copy link
Copy Markdown
Contributor Author

replaces by #8683

@gansheer gansheer closed this May 21, 2026
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.

2 participants