Skip to content
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

.Net: Bump Handlebars.Net from 2.1.4 to 2.1.5 in /dotnet #5719

Merged

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 1, 2024

Bumps Handlebars.Net from 2.1.4 to 2.1.5.

Release notes

Sourced from Handlebars.Net's releases.

2.1.5

Changes

Features 🚀

  • Add EmbedUntrackedSources @​lahma (#570)

    I would also suggest changing to use newer GH Actions images for building so that other warnings would go away (old SDK in use). Maybe another modernization step could be removing old unsupported full framework targets and only support oldest supported net462. Adding net6.0 target would allow one target without dependency on Microsoft.Csharp.

  • Use PackageLicenseExpression in NuGet package @​StefH (#564)

Bug Fixes 🐛

  • Introduce PartialRecursionDepthLimit @​RoosterDragon (#552)

    When evaluating templates with partials, it is possible to recurse in the evaluation of those partials. This can be useful for dealing with tree like data, such as rendering a list of friends-of-friends-of-friends-of-etc....

    The ability to recurse can lead to stack overflows. For example if a sufficiently deep tree is provided as input data, or more simply if the partial calls itself in an infinite loop. As a stack overflow terminates the process, this is not desirable behaviour as it is an unavoidable crash.

    To resolve this a configurable PartialRecursionDepthLimit is introduced, defaulting to 100. Now when a template is evaluated a HandlebarsRuntimeException will be thrown if this limit is reached. This allows the caller to catch the exception and recover gracefully, rather than terminating the process.

  • Allow slashes properly within escape blocks @​Hoeksema (#567)

    closes #566

    The path parsing currently doesn't work properly when there are embedded slashes within an ignore block.

    This PR fixes this issue:

    • No more exceptions thrown when using // within an escaped block
    • Allowing multiple / to occur within an escape block without breakage

    Before, the individual segments between slashes in addition to the entire escaped block were returned by PathInfo. Now, it returns just the latter, which is correct. All existing unit tests still pass and new tests were added to exercise the failing cases in #566.

  • Throw properly on open ignore block instead of crashing @​Hoeksema (#569)

    Closes #568

    Resolve the hang on compile when there is an open ignore block

    Reshuffle the logic so that the throw check for end of template is done before trying to process the char

... (truncated)

Commits
  • bed0c0e Merge pull request #570 from lahma/license-expression
  • 0c6a1ad Merge branch 'master' into license-expression
  • 80727a7 Merge pull request #576 from Handlebars-Net/fix/ci
  • 985e854 Update pull_request.yml
  • e849eab Update ci.yml
  • 4f699a2 Delete .github/FUNDING.yml
  • f3fd1ef Merge pull request #552 from RoosterDragon/recursion-limit
  • c727adf Merge branch 'master' into recursion-limit
  • 9fc63f8 Merge pull request #567 from Hoeksema/patch-1
  • d214f53 Merge branch 'master' into recursion-limit
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [Handlebars.Net](https://github.com/Handlebars-Net/Handlebars.Net) from 2.1.4 to 2.1.5.
- [Release notes](https://github.com/Handlebars-Net/Handlebars.Net/releases)
- [Commits](Handlebars-Net/Handlebars.Net@2.1.4...2.1.5)

---
updated-dependencies:
- dependency-name: Handlebars.Net
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot requested a review from a team as a code owner April 1, 2024 22:54
@dependabot dependabot bot added .NET Issue or Pull requests regarding .NET code dependencies Pull requests that update a dependency file. Used by Dependabot. labels Apr 1, 2024
@github-actions github-actions bot changed the title Bump Handlebars.Net from 2.1.4 to 2.1.5 in /dotnet .Net: Bump Handlebars.Net from 2.1.4 to 2.1.5 in /dotnet Apr 1, 2024
@markwallace-microsoft markwallace-microsoft added this pull request to the merge queue Apr 2, 2024
Merged via the queue into main with commit e383a44 Apr 2, 2024
18 checks passed
@markwallace-microsoft markwallace-microsoft deleted the dependabot/nuget/dotnet/Handlebars.Net-2.1.5 branch April 2, 2024 11:04
LudoCorporateShark pushed a commit to LudoCorporateShark/semantic-kernel that referenced this pull request Aug 25, 2024
)

Bumps [Handlebars.Net](https://github.com/Handlebars-Net/Handlebars.Net)
from 2.1.4 to 2.1.5.
<details>
<summary>Release notes</summary>
<p><em>Sourced from <a
href="https://github.com/Handlebars-Net/Handlebars.Net/releases">Handlebars.Net's
releases</a>.</em></p>
<blockquote>
<h2>2.1.5</h2>
<h2>Changes</h2>
<ul>
<li>
<h3><a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/574">#574</a>
Repo readme is mirrored into nuget package <a
href="https://github.com/thompson-tomo"><code>@​thompson-tomo</code></a>
(<a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/575">#575</a>)</h3>
Closes <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/574">#574</a></li>
</ul>
<h2>Features 🚀</h2>
<ul>
<li>
<h3>Add EmbedUntrackedSources <a
href="https://github.com/lahma"><code>@​lahma</code></a> (<a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/570">#570</a>)</h3>
<ul>
<li>added <code>EmbedUntrackedSources</code> to remove one warning, see
<a
href="https://nuget.info/packages/Handlebars.Net/2.1.4">https://nuget.info/packages/Handlebars.Net/2.1.4</a></li>
</ul>
<p>I would also suggest changing to use newer GH Actions images for
building so that other warnings would go away (old SDK in use). Maybe
another modernization step could be removing old unsupported full
framework targets and only support oldest supported <code>net462</code>.
Adding <code>net6.0</code> target would allow one target without
dependency on <code>Microsoft.Csharp</code>.</p>
</li>
<li>
<h3>Use PackageLicenseExpression in NuGet package <a
href="https://github.com/StefH"><code>@​StefH</code></a> (<a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/564">#564</a>)</h3>
</li>
</ul>
<h2>Bug Fixes 🐛</h2>
<ul>
<li>
<h3>Introduce PartialRecursionDepthLimit <a
href="https://github.com/RoosterDragon"><code>@​RoosterDragon</code></a>
(<a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/552">#552</a>)</h3>
<p>When evaluating templates with partials, it is possible to recurse in
the evaluation of those partials. This can be useful for dealing with
tree like data, such as rendering a list of
friends-of-friends-of-friends-of-etc....</p>
<p>The ability to recurse can lead to stack overflows. For example if a
sufficiently deep tree is provided as input data, or more simply if the
partial calls itself in an infinite loop. As a stack overflow terminates
the process, this is not desirable behaviour as it is an unavoidable
crash.</p>
<p>To resolve this a configurable PartialRecursionDepthLimit is
introduced, defaulting to 100. Now when a template is evaluated a
HandlebarsRuntimeException will be thrown if this limit is reached. This
allows the caller to catch the exception and recover gracefully, rather
than terminating the process.</p>
</li>
<li>
<h3>Allow slashes properly within escape blocks <a
href="https://github.com/Hoeksema"><code>@​Hoeksema</code></a> (<a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/567">#567</a>)</h3>
<p>closes <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/566">#566</a></p>
<p>The path parsing currently doesn't work properly when there are
embedded slashes within an ignore block.</p>
<p>This PR fixes this issue:</p>
<ul>
<li>No more exceptions thrown when using <code>//</code> within an
escaped block</li>
<li>Allowing multiple <code>/</code> to occur within an escape block
without breakage</li>
</ul>
<p>Before, the individual segments between slashes in addition to the
entire escaped block were returned by PathInfo. Now, it returns just the
latter, which is correct. All existing unit tests still pass and new
tests were added to exercise the failing cases in <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/566">#566</a>.</p>
</li>
<li>
<h3>Throw properly on open ignore block instead of crashing <a
href="https://github.com/Hoeksema"><code>@​Hoeksema</code></a> (<a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/569">#569</a>)</h3>
<p>Closes <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/568">#568</a></p>
<p>Resolve the hang on compile when there is an open ignore block</p>
<p>Reshuffle the logic so that the throw check for end of template is
done before trying to process the char</p>
</li>
</ul>
<!-- raw HTML omitted -->
</blockquote>
<p>... (truncated)</p>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/bed0c0e8fca230de68243c43843bb23cb4e0ae0d"><code>bed0c0e</code></a>
Merge pull request <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/570">#570</a>
from lahma/license-expression</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/0c6a1ad6cedfaf63a4ee4149770bb4e0f15e2826"><code>0c6a1ad</code></a>
Merge branch 'master' into license-expression</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/80727a770a807346b5f6730d19434e5c92d0d8fc"><code>80727a7</code></a>
Merge pull request <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/576">#576</a>
from Handlebars-Net/fix/ci</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/985e854be81d38f0685d5e23093a241acfb08443"><code>985e854</code></a>
Update pull_request.yml</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/e849eab88e76664efd04d125589a6d5806262653"><code>e849eab</code></a>
Update ci.yml</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/4f699a269b50dc9b5d7e71cee5c81a44abd4aaa6"><code>4f699a2</code></a>
Delete .github/FUNDING.yml</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/f3fd1ef1b68a4f762fee8dfa19477820cba5579b"><code>f3fd1ef</code></a>
Merge pull request <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/552">#552</a>
from RoosterDragon/recursion-limit</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/c727adfc69811807574ef2abc00f9c73d38c4683"><code>c727adf</code></a>
Merge branch 'master' into recursion-limit</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/9fc63f8118a7263b26bb73f242ea8cac5b27b053"><code>9fc63f8</code></a>
Merge pull request <a
href="https://redirect.github.com/Handlebars-Net/Handlebars.Net/issues/567">#567</a>
from Hoeksema/patch-1</li>
<li><a
href="https://github.com/Handlebars-Net/Handlebars.Net/commit/d214f53da8d4285a3077bd0b508ad5ac41e6152b"><code>d214f53</code></a>
Merge branch 'master' into recursion-limit</li>
<li>Additional commits viewable in <a
href="https://github.com/Handlebars-Net/Handlebars.Net/compare/2.1.4...2.1.5">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=Handlebars.Net&package-manager=nuget&previous-version=2.1.4&new-version=2.1.5)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after
your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge
and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating
it. You can achieve the same result by closing it manually
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Roger Barreto <19890735+RogerBarreto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file. Used by Dependabot. .NET Issue or Pull requests regarding .NET code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants