-
Notifications
You must be signed in to change notification settings - Fork 75
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
feat(stdlib): add encode_charset, decode_charset functions #1162
Conversation
Charset encoding and decoding functions were implemented to integrate the charset settings used in various operating systems and systems with vector. - decode_charset(value, from_charset) - encode_charset(value, to_charset)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @powerumc!
Left a couple of minor comments.
- rename `error` function to `create_error` - rename `from` to `from_charset` in decode_charset - rename `from` to `to_charset` in encode_charset
e72060f
to
d4be5d7
Compare
@pront |
The licenses check fails, please use |
In my case, there is no change in the The error occurs in the check-deny step. Is this caused by my code? |
I think we just needed |
Summary
Charset encoding and decoding functions were implemented to integrate the charset settings used in various operating systems and systems with vector.
Change Type
Is this a breaking change?
How did you test this PR?
Does this PR include user facing changes?
our guidelines.
Checklist
run
dd-rust-license-tool write
and commit the changes. More details here.PR adding new VRL function:
PR adding documentation:
References