Skip to content

Conversation

@pditommaso
Copy link
Member

Summary

  • Escape % characters to %% in error messages thrown by HttpPluginRepository when a connection exception occurs
  • This prevents String.format interpretation errors in PluginRuntimeException when the URI contains URL-encoded characters (e.g., %20 for spaces)
  • Added test case to verify error messages with percent-encoded URLs are handled correctly

Test plan

  • New test case handle prefetch error with percent chars in error message passes
  • All existing HttpPluginRepositoryTest tests pass

🤖 Generated with Claude Code

Escape '%' characters to '%%' in the error message when a connection
exception occurs in HttpPluginRepository. This prevents String.format
interpretation errors in PluginRuntimeException when the URI contains
URL-encoded characters (e.g., %20 for spaces).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@netlify
Copy link

netlify bot commented Dec 11, 2025

Deploy Preview for nextflow-docs-staging canceled.

Name Link
🔨 Latest commit 45b6bc6
🔍 Latest deploy log https://app.netlify.com/projects/nextflow-docs-staging/deploys/693c4d3820b3a800076e3fa3

…ry.groovy [ci fast]

Co-authored-by: Ben Sherman <bentshermann@gmail.com>
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
@pditommaso pditommaso merged commit 59c4f4e into master Dec 12, 2025
14 checks passed
@pditommaso pditommaso deleted the fix/escape-percent-in-plugin-error-message branch December 12, 2025 17:27
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