Rasterize Latest Poppler #32997
Merged
Rasterize Latest Poppler #32997
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Coverage Report
|
||||||||||||||||||||||||||||||
anara123
reviewed
Feb 20, 2024
|
|
||
| def test_poppler_version(): | ||
| import pdf2image | ||
| poppler_version = pdf2image.pdf2image._get_poppler_version("pdftoppm") |
There was a problem hiding this comment.
lets have an e2e test like we have in ParseEmailFiles where we use test files and run the command and check for the output
There was a problem hiding this comment.
Poppler is a C executable, and pdf2image is a Python wrapper.
In order to reproduce the issue I'll need a DocuSign doc, which we don't have.
tkatzir
added
the
ForceMerge
|
Needs to be force-merged due to unrelated build issues. |
anara123
approved these changes
Mar 3, 2024
… into rasterize-latest-poppler
MosheEichler
pushed a commit
that referenced
this pull request
Mar 4, 2024
MosheEichler
added a commit
that referenced
this pull request
Mar 21, 2024
…ation (#33155) * fix default key * rn * docker * docs review * [ConvertFile] Bump Docker version (#33141) * Update Docker version * Bump version * ruff * Bump pack from version CommonScripts to 1.14.4. * Bump Docker version * Bump pack from version CommonScripts to 1.14.5. * Bump pack from version CommonScripts to 1.14.6. --------- Co-authored-by: Content Bot <[email protected]> * Added Compatibility with new Allowed Indicator Module in CTIX 3.6 (#33062) (#33159) I also ran all of the pre-commit hooks and resolved all of those issues. Co-authored-by: Corey Bodendein <[email protected]> * Fix/[XSUP-33795]/Logo does not shown in data collection form (#33154) * change default to true * add s flag * add UT * RN * pre-commit * mocker * fix default key * revert * EWS v2: added arg to the command (#33096) * added arg to the command * Apply suggestions from code review Co-authored-by: ShirleyDenkberg <[email protected]> * fix validate --------- Co-authored-by: ShirleyDenkberg <[email protected]> * Rasterize Latest Poppler (#32997) * Teams generate url command bug (#33164) * fixed a bug in generate url command * added rn * added rn * edit * added bc rn * edits * fixed unit test * updated do * Enhancement | InfoBlox - add list host and network info, modify get IP cmds (#32619) * added host records * handle no comma,equal sign in extattrs * wip: added network,to/from_ip args to get_ip_cmd * added validation for invalid input args to get-ip moved request params into client * added max_results to list host cmd * added network info cmd * modified get-ip command to return a list added rns * format/validation fixes * fixed context output for host records cmd * fixed output table for net info cmd * added additional return fields to list host args parsed extattrs into dict for context * updated integration docs added command examples fixed context key in net info command added types to context outputs for list host and net cmds * fixed netview context key in net uts * changed context output types for get-ip added bc mentions for get-ip context output keys * fixed hardcoded default values in args description add breaking changes in md * mv test_files test_data * mv test_files test_data * converted integration readme from html to markdown pr review: empty dict evaluates to false, no need to cast to bool * strip whitespace from get-ip context path in readme * add unique to ipv4address status client params instantiated with _return_as_object=1 add capture stderr ignore pr review: use any() instead of sum() == 0 * pr review: rm endpoint suffixes from client pr review: rm max_results from list host commandres title pr review: add default val for max results to bc * pr review: rm ip,netmask input validation rm max_results and added mode/val in title for get-ip-cmd * mv request params into client methods * rm transform_return_fields * simplified raw to context mapping * raise exception when extattrs input is invalid * updated docker image rm max results from hr for net info cmd added todos for checking unexpected invalid extattrs rm exception var * fixed issue with additional return fields * fix flake8 errs * rm todos fixed issue with too many args supplied to ip command * revert changes to too many args handle in ip cmd * fixed titles for get-ip, get host info, get network info * added rn comment about get-ip mod outputs * fix rn comment about get-ip mod outputs * fix cmd outputs assertions in uts * added args to get-ip in integration doc * update docker image * fix docs * revert poetry * fix docs * Update README.md (#33211) edited description of cron argument, per Dudu B. * download and move images to content repo (#33204) * download and move images to content repo * commit test * revert changes for gitlab_slack_notifier * Update Docker Image To demisto/chromium (#33175) * Updated Metadata Of Pack rasterize * Added release notes to pack rasterize * Packs/rasterize/Integrations/rasterize/rasterize.yml Docker image update * Updated Metadata Of Pack ExpanseV2 * Added release notes to pack ExpanseV2 * Packs/ExpanseV2/Scripts/ExpanseGenerateIssueMapWidgetScript/ExpanseGenerateIssueMapWidgetScript.yml Docker image update * Bump pack from version ExpanseV2 to 1.10.51. * Bump pack from version rasterize to 2.0.2. --------- Co-authored-by: Content Bot <[email protected]> * docker * do not write files when running test-module * RN * ignore error 125 * docker * Fix RN * revert * ignore investigation attachments in test module command --------- Co-authored-by: Michael Yochpaz <[email protected]> Co-authored-by: Content Bot <[email protected]> Co-authored-by: content-bot <[email protected]> Co-authored-by: Corey Bodendein <[email protected]> Co-authored-by: Israel Lappe <[email protected]> Co-authored-by: ShirleyDenkberg <[email protected]> Co-authored-by: tkatzir <[email protected]> Co-authored-by: merit-maita <[email protected]> Co-authored-by: Kobbi Gal <[email protected]> Co-authored-by: Tamar Schanfeld <[email protected]> Co-authored-by: israelpoli <[email protected]>
pal-xmco
pushed a commit
to pal-xmco/content
that referenced
this pull request
Jun 19, 2024
…ation (demisto#33155) * fix default key * rn * docker * docs review * [ConvertFile] Bump Docker version (demisto#33141) * Update Docker version * Bump version * ruff * Bump pack from version CommonScripts to 1.14.4. * Bump Docker version * Bump pack from version CommonScripts to 1.14.5. * Bump pack from version CommonScripts to 1.14.6. --------- Co-authored-by: Content Bot <[email protected]> * Added Compatibility with new Allowed Indicator Module in CTIX 3.6 (demisto#33062) (demisto#33159) I also ran all of the pre-commit hooks and resolved all of those issues. Co-authored-by: Corey Bodendein <[email protected]> * Fix/[XSUP-33795]/Logo does not shown in data collection form (demisto#33154) * change default to true * add s flag * add UT * RN * pre-commit * mocker * fix default key * revert * EWS v2: added arg to the command (demisto#33096) * added arg to the command * Apply suggestions from code review Co-authored-by: ShirleyDenkberg <[email protected]> * fix validate --------- Co-authored-by: ShirleyDenkberg <[email protected]> * Rasterize Latest Poppler (demisto#32997) * Teams generate url command bug (demisto#33164) * fixed a bug in generate url command * added rn * added rn * edit * added bc rn * edits * fixed unit test * updated do * Enhancement | InfoBlox - add list host and network info, modify get IP cmds (demisto#32619) * added host records * handle no comma,equal sign in extattrs * wip: added network,to/from_ip args to get_ip_cmd * added validation for invalid input args to get-ip moved request params into client * added max_results to list host cmd * added network info cmd * modified get-ip command to return a list added rns * format/validation fixes * fixed context output for host records cmd * fixed output table for net info cmd * added additional return fields to list host args parsed extattrs into dict for context * updated integration docs added command examples fixed context key in net info command added types to context outputs for list host and net cmds * fixed netview context key in net uts * changed context output types for get-ip added bc mentions for get-ip context output keys * fixed hardcoded default values in args description add breaking changes in md * mv test_files test_data * mv test_files test_data * converted integration readme from html to markdown pr review: empty dict evaluates to false, no need to cast to bool * strip whitespace from get-ip context path in readme * add unique to ipv4address status client params instantiated with _return_as_object=1 add capture stderr ignore pr review: use any() instead of sum() == 0 * pr review: rm endpoint suffixes from client pr review: rm max_results from list host commandres title pr review: add default val for max results to bc * pr review: rm ip,netmask input validation rm max_results and added mode/val in title for get-ip-cmd * mv request params into client methods * rm transform_return_fields * simplified raw to context mapping * raise exception when extattrs input is invalid * updated docker image rm max results from hr for net info cmd added todos for checking unexpected invalid extattrs rm exception var * fixed issue with additional return fields * fix flake8 errs * rm todos fixed issue with too many args supplied to ip command * revert changes to too many args handle in ip cmd * fixed titles for get-ip, get host info, get network info * added rn comment about get-ip mod outputs * fix rn comment about get-ip mod outputs * fix cmd outputs assertions in uts * added args to get-ip in integration doc * update docker image * fix docs * revert poetry * fix docs * Update README.md (demisto#33211) edited description of cron argument, per Dudu B. * download and move images to content repo (demisto#33204) * download and move images to content repo * commit test * revert changes for gitlab_slack_notifier * Update Docker Image To demisto/chromium (demisto#33175) * Updated Metadata Of Pack rasterize * Added release notes to pack rasterize * Packs/rasterize/Integrations/rasterize/rasterize.yml Docker image update * Updated Metadata Of Pack ExpanseV2 * Added release notes to pack ExpanseV2 * Packs/ExpanseV2/Scripts/ExpanseGenerateIssueMapWidgetScript/ExpanseGenerateIssueMapWidgetScript.yml Docker image update * Bump pack from version ExpanseV2 to 1.10.51. * Bump pack from version rasterize to 2.0.2. --------- Co-authored-by: Content Bot <[email protected]> * docker * do not write files when running test-module * RN * ignore error 125 * docker * Fix RN * revert * ignore investigation attachments in test module command --------- Co-authored-by: Michael Yochpaz <[email protected]> Co-authored-by: Content Bot <[email protected]> Co-authored-by: content-bot <[email protected]> Co-authored-by: Corey Bodendein <[email protected]> Co-authored-by: Israel Lappe <[email protected]> Co-authored-by: ShirleyDenkberg <[email protected]> Co-authored-by: tkatzir <[email protected]> Co-authored-by: merit-maita <[email protected]> Co-authored-by: Kobbi Gal <[email protected]> Co-authored-by: Tamar Schanfeld <[email protected]> Co-authored-by: israelpoli <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Status
Related Issues
fixes: https://jira-dc.paloaltonetworks.com/browse/XSUP-32892