Skip to content

FOUR-19373: Incorrect validation - Error uploading PDF file: Dangerous content of PDF file#7417

Merged
ryancooley merged 1 commit intonextfrom
bugfix/FOUR-19373
Sep 29, 2024
Merged

FOUR-19373: Incorrect validation - Error uploading PDF file: Dangerous content of PDF file#7417
ryancooley merged 1 commit intonextfrom
bugfix/FOUR-19373

Conversation

@danloa
Copy link
Copy Markdown
Contributor

@danloa danloa commented Sep 25, 2024

Issue & Reproduction Steps

Create a screen with a file upload feature.

Integrate the screen into a process.

Upload the next PDF

student.pdf

Current behavior:
The PDF is blocked

Expected behavior
PDFs that don't have Javascript should be uploaded to ProcessMaker

Solution

  • Modified the conditions to validate a PDF

Related Tickets & Packages

- Link to any related FOUR tickets, PRDs, or packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

ci:next
ci:deploy

@eiresendez eiresendez self-requested a review September 26, 2024 14:54
@tokensPM
Copy link
Copy Markdown

QA server K8S was successfully deployed https://ci-dc721e3457.engk8s.processmaker.net

@ryancooley ryancooley merged commit 319f16f into next Sep 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants