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

Sanitize HTML in Event Descriptions #194

Conversation

ThorntonMatthewD
Copy link
Collaborator

@ThorntonMatthewD ThorntonMatthewD commented Dec 11, 2023

WIP

Adds the mews/purifier package as a depdency and utilizes it to sanitize HTML contained in event descriptions prior to persisting them.

Screenshot_2023-12-10_21-55-44

@bogdankharchenko
Copy link
Collaborator

Looks good!

@allella
Copy link
Member

allella commented Dec 11, 2023

@ThorntonMatthewD is there a configuration file that's being set for this package? If so, would we want to add something to the .env.example or some other place so we know what the config should look like?

@ThorntonMatthewD
Copy link
Collaborator Author

@allella So far I've just been tinkering with the default config that the package ships with. We could specify a custom config at config/purifier.php if the need arises, but I've not done enough testing to determine if that is something that should be pursued.

@allella
Copy link
Member

allella commented Dec 11, 2023

Sounds good.

@bogdankharchenko
Copy link
Collaborator

@ThorntonMatthewD let's get this merged in

@bogdankharchenko
Copy link
Collaborator

@ThorntonMatthewD Can we get this merged in? =)

@allella
Copy link
Member

allella commented Apr 12, 2024

@bogdankharchenko did we address this in another way?

The HTML in the Slackbot in #events does some goofy stuff, but that's perhaps a problem to be solved in that repo.

@bogdankharchenko
Copy link
Collaborator

I will solve it another way

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.

3 participants