Skip to content

Add spring boot 4 trailing slash migration#2034

Open
gdgenchev wants to merge 1 commit into
github:stagedfrom
gdgenchev:staged
Open

Add spring boot 4 trailing slash migration#2034
gdgenchev wants to merge 1 commit into
github:stagedfrom
gdgenchev:staged

Conversation

@gdgenchev

@gdgenchev gdgenchev commented Jun 17, 2026

Copy link
Copy Markdown

Pull Request Checklist

  • I have read and followed the CONTRIBUTING.md guidelines.
  • I have read and followed the Guidance for submissions involving paid services.
  • My contribution adds a new instruction, prompt, agent, skill, workflow, or canvas extension file in the correct directory.
  • The file follows the required naming convention.
  • The content is clearly structured and follows the example format.
  • I have tested my instructions, prompt, agent, skill, workflow, or canvas extension with GitHub Copilot.
  • I have run npm start and verified that README.md is up to date.
  • I am targeting the staged branch for this pull request.

Description

I bumped into this issue with trailing slash config removal in spring boot 4 that was not properly handled by open-rewrite recipe, because it was set dynamically (based on a flag). The open-rewrite recipe just deleted the setting, which is dangerous if it remains unnoticed and reaches production and if other services depend on this.


Type of Contribution

  • New instruction file.
  • New prompt file.
  • New agent file.
  • New plugin.
  • New skill file.
  • New agentic workflow.
  • New canvas extension.
  • Update to existing instruction, prompt, agent, plugin, skill, workflow, or canvas extension.
  • Other (please specify):

By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.

@gdgenchev gdgenchev requested a review from aaronpowell as a code owner June 17, 2026 12:59
@github-actions github-actions Bot added the instructions PR touches instructions label Jun 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔒 PR Risk Scan Results

Scanned 1 changed file(s).

Severity Count
🔴 High 0
🟠 Medium 0
ℹ️ Info 0

✅ No matching risk patterns were detected in changed files.

This is an automated soft-gate report. Findings indicate review targets and do not block merge by themselves.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

instructions PR touches instructions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant