Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

fix($sce): fix adjustMatcher to replace multiple '*' and '**' #7897

Closed
wants to merge 1 commit into from
Closed

fix($sce): fix adjustMatcher to replace multiple '*' and '**' #7897

wants to merge 1 commit into from

Conversation

santialbo
Copy link
Contributor

On $sceDelegateProvide the adjustMatcher function was only replacing the first * and ** found on the whitelisted and blacklisted urls. This caused $sceDelegate policy problems for example when trying to match *://*.example.com/** against https://www.example.com/path.

adjustMatcher was only replacing the first '*' and '**' found on the whitelisted
and blacklisted urls.
@mary-poppins
Copy link

I'm sorry, but I wasn't able to verify your Contributor License Agreement (CLA) signature. CLA signature is required for any code contributions to AngularJS.

Please sign our CLA and ensure that the CLA signature email address and the email address in this PR's commits match.

If you signed the CLA as a corporation, please let us know the company's name.

Thanks a bunch!

PS: If you signed the CLA in the past then most likely the email addresses don't match. Please sign the CLA again or update the email address in the commit of this PR.
PS2: If you are a Googler, please sign the CLA as well to simplify the CLA verification process.

@santialbo
Copy link
Contributor Author

I signed the CLA again (I did already in the past with the same email address)

@mary-poppins
Copy link

CLA signature verified! Thank you!

Someone from the team will now triage your PR and it will be processed based on the determined priority (doc updates and fixes with tests are prioritized over other changes).

@rodyhaddad rodyhaddad added this to the Backlog milestone Jun 23, 2014
@santialbo
Copy link
Contributor Author

More context on here. It doesn't mention that you can use wildcards only once.

@santialbo
Copy link
Contributor Author

I ran into this issue again. The workaround is straighforward but merging this would be nice given that it's such a small fix.

@Narretz Narretz added this to the 1.6.x milestone Jan 24, 2016
@Narretz Narretz removed this from the Backlog milestone Jan 24, 2016
petebacondarwin pushed a commit that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes ##7897
petebacondarwin pushed a commit that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes ##7897
@santialbo santialbo deleted the fix-adjust-matchers branch November 21, 2016 15:16
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes #angular#7897
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes #angular#7897
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes #angular#7897
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes #angular#7897
petebacondarwin pushed a commit to petebacondarwin/angular.js that referenced this pull request Nov 21, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes #angular#7897
petebacondarwin pushed a commit that referenced this pull request Nov 23, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes ##7897
petebacondarwin pushed a commit that referenced this pull request Nov 24, 2016
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes ##7897
ellimist pushed a commit to ellimist/angular.js that referenced this pull request Mar 15, 2017
`adjustMatcher` was only replacing the first occurrences of '*' and '**'
that were found in whitelisted and blacklisted url strings.

Closes #angular#7897
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants