Skip to content

7.1.x url mapping greedy extension 3#15522

Open
codeconsole wants to merge 2 commits intoapache:7.1.xfrom
codeconsole:7.1.x-url-mapping-greedy-extension-3
Open

7.1.x url mapping greedy extension 3#15522
codeconsole wants to merge 2 commits intoapache:7.1.xfrom
codeconsole:7.1.x-url-mapping-greedy-extension-3

Conversation

@codeconsole
Copy link
Contributor

@codeconsole codeconsole commented Mar 21, 2026

Final fix for greedy url mapping parameter

Verified with:

        // /scott.smith.json, /scott.json, /12345.json, /scott, /12345
        "/${id}+(.$format)?"(controller:'user', action:'profile') { 
            constraints {
                format(inList: ['json', 'xml'])
            }
        }
        // Fallback for IDs with dots but no valid format (e.g. /scott.smith → id=scott.smith)
        "/${id}"(controller:'user', action:'profile')

I had found another edge case that wasn't working. This PR should get the job done.

@codeconsole codeconsole changed the base branch from 7.0.x to 7.1.x March 21, 2026 01:15
@codeconsole
Copy link
Contributor Author

#15213
#15358

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

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant