Skip to content

fix(core): CSS sanitizer now allows parens in file names #30322

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

benlesh
Copy link
Contributor

@benlesh benlesh commented May 8, 2019

Resolves an issue where images that were created with a name like 'foo (1).png' would not pass CSS url sanitization.

Resolves an issue where images that were created with a name like `'foo (1).png'` would not pass CSS url sanitization.
@benlesh benlesh added type: bug/fix freq1: low area: core Issues related to the framework runtime target: patch This PR is targeted for the next patch release risk: low labels May 8, 2019
@ngbot ngbot bot modified the milestone: needsTriage May 8, 2019
@benlesh benlesh marked this pull request as ready for review May 8, 2019 02:51
@benlesh benlesh requested review from a team as code owners May 8, 2019 02:51
@benlesh
Copy link
Contributor Author

benlesh commented May 8, 2019

presubmit

@benlesh benlesh added the action: merge The PR is ready for merge by the caretaker label May 8, 2019
@alxhub
Copy link
Member

alxhub commented May 8, 2019

Working presubmit link

@alxhub
Copy link
Member

alxhub commented May 8, 2019

A better presubmit

alxhub pushed a commit that referenced this pull request May 8, 2019
Resolves an issue where images that were created with a name like `'foo (1).png'` would not pass CSS url sanitization.

PR Close #30322
@alxhub alxhub closed this May 8, 2019
alxhub pushed a commit that referenced this pull request May 8, 2019
Resolves an issue where images that were created with a name like `'foo (1).png'` would not pass CSS url sanitization.

PR Close #30322
alxhub added a commit to alxhub/angular that referenced this pull request May 14, 2019
…ular#30322)"

This reverts commit 728db88.

We're reverting this commit for now, until it can be subjected to a more
thorough security review.
alxhub added a commit that referenced this pull request May 14, 2019
)" (#30463)

This reverts commit 728db88.

We're reverting this commit for now, until it can be subjected to a more
thorough security review.

PR Close #30463
alxhub added a commit that referenced this pull request May 14, 2019
)" (#30463)

This reverts commit 728db88.

We're reverting this commit for now, until it can be subjected to a more
thorough security review.

PR Close #30463
BioPhoton pushed a commit to BioPhoton/angular that referenced this pull request May 21, 2019
Resolves an issue where images that were created with a name like `'foo (1).png'` would not pass CSS url sanitization.

PR Close angular#30322
BioPhoton pushed a commit to BioPhoton/angular that referenced this pull request May 21, 2019
…ular#30322)" (angular#30463)

This reverts commit 728db88.

We're reverting this commit for now, until it can be subjected to a more
thorough security review.

PR Close angular#30463
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker area: core Issues related to the framework runtime cla: yes freq1: low risk: low target: patch This PR is targeted for the next patch release type: bug/fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants