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

Do not use default escaping on address display #22120

Merged
merged 1 commit into from
Nov 24, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

Do not use default escaping on address display

This has vcard html in it and so breaks if default escaping is enabled

Before

Broken html with escape-on-output enabled

After

Output not subject to default escaping, purify added as good practice while doing this on any field that could contain inputted datat

Technical Details

Comments

This has vcard html in it and so breaks if default escaping is enabled
@civibot
Copy link

civibot bot commented Nov 23, 2021

(Standard links)

@civibot civibot bot added the master label Nov 23, 2021
@eileenmcnaughton eileenmcnaughton merged commit 1f611b5 into civicrm:master Nov 24, 2021
@eileenmcnaughton eileenmcnaughton deleted the address branch November 24, 2021 01:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants