Skip to content

FOUR-9018: Define a variable in ENV to turn off the security Logs#4937

Merged
pmPaulis merged 1 commit into
feature/FOUR-8121from
feature/FOUR-9018
Jun 30, 2023
Merged

FOUR-9018: Define a variable in ENV to turn off the security Logs#4937
pmPaulis merged 1 commit into
feature/FOUR-8121from
feature/FOUR-9018

Conversation

@pmPaulis
Copy link
Copy Markdown
Contributor

@pmPaulis pmPaulis commented Jun 28, 2023

Issue & Reproduction Steps

Register some events related to user activity events according to the PRD.
Define a variable in ENV to turn off the security Logs

Solution

  • Improve the tickets definition

How to Test

if the user needs to disable the security log needs to define in the ENV

SECURITY_LOG = false;

Related Tickets & 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:deploy

@pmPaulis pmPaulis requested review from caleeli and fagubla June 28, 2023 19:19
@pmPaulis pmPaulis force-pushed the feature/FOUR-9018 branch from 778b0c8 to c27c246 Compare June 28, 2023 19:52
@pmkerberos
Copy link
Copy Markdown

QA server was successfully deployed https://ci-949b2ffaba.eng.processmaker.net

@pmPaulis pmPaulis force-pushed the feature/FOUR-9018 branch from c27c246 to bd2e8ca Compare June 29, 2023 00:16
@pmkerberos
Copy link
Copy Markdown

QA server was successfully deployed https://ci-949b2ffaba.eng.processmaker.net

@pmPaulis pmPaulis force-pushed the feature/FOUR-9018 branch from bd2e8ca to 72dc419 Compare June 29, 2023 00:55
@pmPaulis pmPaulis force-pushed the feature/FOUR-9018 branch from 72dc419 to d1fb299 Compare June 29, 2023 20:07
@pmPaulis pmPaulis force-pushed the feature/FOUR-9018 branch from d1fb299 to d5514d1 Compare June 29, 2023 20:14
@pmPaulis pmPaulis merged commit 8dec864 into feature/FOUR-8121 Jun 30, 2023
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

Successfully merging this pull request may close these issues.

4 participants