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

[verbosity] [security] Escape or strip terminal/control codes when printing to a terminal with variables #4

Open
chl0e3e opened this issue Aug 29, 2022 · 2 comments

Comments

@chl0e3e
Copy link

chl0e3e commented Aug 29, 2022

Code
Code
Code
Code
Code
Code
Code
Code
Code
Code
Code

@gizmore
Copy link
Owner

gizmore commented Sep 1, 2022

Another take to utilize github.com issues 👍

It is unclear to me what you mean!

BUT It is important to have none, or as few as possible, filters and mutating transformers.

If there are two spaces, - i want those two spaces!

Example:

bash $ gdo mail.send chl0,Hi      theeeeeeere,I wonder which mail clients keep these spaces.

This command actually should send a mail to you, or crashes.

I am working on the renderings, maybe this issue got fixed in a coding spree?

Thanks again for taking your time!

  • giz

@gizmore
Copy link
Owner

gizmore commented Feb 25, 2023

Sadly all the Line numbers are off now, but i assume you mean an impl for the html() function (which is kinda htmlspecialchars in www rendering mode, and current NOOP in CLI)
A solution would be to implement a CLI behaviour in said function.
@todo: maybe make the html() function a callback to remove one or two IF's

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

No branches or pull requests

2 participants