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
Encoding guesser #1472
Encoding guesser #1472
Conversation
70583ac
to
759a0b6
Compare
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.
minor nits, can we refactor the guesser to its own package? else LGTM!
Hey @m-brophy ! Just checking back on this! The PR looks almost ready to merge with some minor nits. |
1e0c59f
to
aa20c8a
Compare
Hi @lumjjb, changes made, branch rebased, CI tests all successful if you want to take another look now. |
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
Signed-off-by: m-brophy <mbrophy@redhat.com>
72ef396
to
2f157a4
Compare
Hey @lumjjb I just rebased this ready to merge if you're happy to do that |
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.
LGTM! Thanks @m-brophy
Signed-off-by: Jeff Mendoza <jlm@jlm.name>
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.
Addressed some minor style nits.
Description of the PR
This PR enables guac to detect the encoding of a compressed file from the internal bytes of the document, for use if this information isn't supplied separately
Fixes #1237
http.DetectContentType
is used but this does not handlebz2
orzst
encoding so this capability has been addedthese docs were used as a reference for detection of these types: https://www.ietf.org/rfc/rfc8878.txt and https://www.ietf.org/rfc/rfc5655.txt
PR Checklist
-s
flag togit commit
.make generate
has been runcollectsub
protobuf has been changed,make proto
has been run