Skip to content

fix: Fix issue that could lead to RCE if using unsecure Jinja templates in dynamic prompt builders#8096

Merged
silvanocerza merged 1 commit into
v2.3.xfrom
dynamic-prompt-builder-fix
Jul 26, 2024
Merged

fix: Fix issue that could lead to RCE if using unsecure Jinja templates in dynamic prompt builders#8096
silvanocerza merged 1 commit into
v2.3.xfrom
dynamic-prompt-builder-fix

Conversation

@silvanocerza
Copy link
Copy Markdown
Contributor

Proposed Changes:

Brings fix from #8095 in for dynamic prompt builders.

How did you test it?

I ran tests locally.

Notes for the reviewer

N/A

Checklist

@silvanocerza silvanocerza requested a review from shadeMe July 26, 2024 14:44
@silvanocerza silvanocerza self-assigned this Jul 26, 2024
@silvanocerza silvanocerza requested review from a team as code owners July 26, 2024 14:44
@silvanocerza silvanocerza requested review from dfokina and vblagoje and removed request for a team and vblagoje July 26, 2024 14:44
@silvanocerza silvanocerza force-pushed the dynamic-prompt-builder-fix branch 2 times, most recently from 327dcf1 to 25bd9a7 Compare July 26, 2024 15:47
@silvanocerza silvanocerza force-pushed the dynamic-prompt-builder-fix branch from 25bd9a7 to bff331e Compare July 26, 2024 15:56
@coveralls
Copy link
Copy Markdown
Collaborator

Pull Request Test Coverage Report for Build 10114113783

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.003%) to 90.017%

Totals Coverage Status
Change from base Build 10114108858: 0.003%
Covered Lines: 6862
Relevant Lines: 7623

💛 - Coveralls

@silvanocerza silvanocerza merged commit 6c25a5c into v2.3.x Jul 26, 2024
@silvanocerza silvanocerza deleted the dynamic-prompt-builder-fix branch July 26, 2024 16:17
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.

3 participants