Skip to content
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

Velocity 931 update secure classlist #16

Merged

Conversation

wglasshusain
Copy link
Contributor

No description provided.

@nbubna
Copy link

nbubna commented Aug 6, 2020

Looks good to me.

@pwntester
Copy link

hi @wglasshusain @nbubna , can we get this merged so the new version can be released?

Thanks!

@pwntester
Copy link

pwntester commented Jan 8, 2021

hi @wglasshusain @nbubna, any update on this?

@arkanovicz arkanovicz merged commit f355cec into apache:master Feb 25, 2021
@lishuochuan
Copy link

May I know any plan to merge the fix to velocity-1.7?

@arkanovicz
Copy link
Contributor

There is no plan whatsoever to update 1.7.

Look. Velocity 1.7 was released in 2010. That's more than ten years ago. You can't seriously ask for us to maintain ten years old code. Feel free to integrate the patch yourself, though.

@cesarhernandezgt
Copy link

@lishuochuan @arkanovicz
Hello, I created https://issues.apache.org/jira/browse/VELOCITY-941 with the backport to 1.7.x branch.
I basically cherry-pick the changes from this PR made by @wglasshusain.
I'll follow up in the dev@ mailing list.

mbowerman added a commit to mbowerman/liferay-portal that referenced this pull request Sep 12, 2022
mbowerman added a commit to mbowerman/liferay-portal that referenced this pull request Sep 12, 2022
brianchandotcom pushed a commit to brianchandotcom/liferay-portal that referenced this pull request Sep 19, 2022
mbowerman added a commit to mbowerman/liferay-portal that referenced this pull request Sep 19, 2022
claraizquierdo pushed a commit to liferay-tango/liferay-portal that referenced this pull request Sep 20, 2022
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.

7 participants